[llvm] r319530 - [lit] Don't enable LSan on Darwin for Apple clang 9.0.0

Jonas Devlieghere via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 1 02:49:47 PST 2017


Author: jdevlieghere
Date: Fri Dec  1 02:49:47 2017
New Revision: 319530

URL: http://llvm.org/viewvc/llvm-project?rev=319530&view=rev
Log:
[lit] Don't enable LSan on Darwin for Apple clang 9.0.0

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

Modified:
    llvm/trunk/utils/lit/lit/llvm/config.py

Modified: llvm/trunk/utils/lit/lit/llvm/config.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/llvm/config.py?rev=319530&r1=319529&r2=319530&view=diff
==============================================================================
--- llvm/trunk/utils/lit/lit/llvm/config.py (original)
+++ llvm/trunk/utils/lit/lit/llvm/config.py Fri Dec  1 02:49:47 2017
@@ -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
 




More information about the llvm-commits mailing list