[PATCH] D40672: [lit] Don't enable LSan on darwin for clang 900

[Jonas Devlieghere via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL319530: [lit] Don't enable LSan on Darwin for Apple clang 9.0.0 (authored by JDevlieghere).

Changed prior to commit:
  https://reviews.llvm.org/D40672?vs=124997&id=125092#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D40672

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


Index: llvm/trunk/utils/lit/lit/llvm/config.py
===================================================================
--- llvm/trunk/utils/lit/lit/llvm/config.py
+++ llvm/trunk/utils/lit/lit/llvm/config.py
@@ -223,12 +223,13 @@
             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
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40672.125092.patch
Type: text/x-patch
Size: 946 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171201/95767574/attachment.bin>