]> granicus.if.org Git - llvm/commitdiff
[lit] Don't enable LSan on Darwin for Apple clang 9.0.0
authorJonas Devlieghere <jonas@devlieghere.com>
Fri, 1 Dec 2017 10:49:47 +0000 (10:49 +0000)
committerJonas Devlieghere <jonas@devlieghere.com>
Fri, 1 Dec 2017 10:49:47 +0000 (10:49 +0000)
The latest clang that ships with Xcode (clang 900 or 9.0.0) does not
support LSan. This fixes the lit configuration to reflect that.

Differential revision: https://reviews.llvm.org/D40672

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@319530 91177308-0d34-0410-b5e6-96231b3b80d8

utils/lit/lit/llvm/config.py

index c631f8b8865801bb5756e9db2253e9f4e7cfd225..e3be929d716ded85ce15b4ff0e0c06d1c04436b9 100644 (file)
@@ -223,12 +223,13 @@ class LLVMConfig(object):
             return True
 
         if re.match(r'^x86_64.*-apple', triple):
-            version_number = int(
-                re.search(r'version ([0-9]+)\.', version_string).group(1))
+            version_regex = re.search(r'version ([0-9]+)\.([0-9]+).([0-9]+)', version_string)
+            major_version_number = int(version_regex.group(1))
+            minor_version_number = int(version_regex.group(3))
             if 'Apple LLVM' in version_string:
-                return version_number >= 9
+                return major_version_number >= 9 and minor_version_number > 0
             else:
-                return version_number >= 5
+                return major_version_number >= 5
 
         return False