[PATCH] D49115: Add lowercase OS name feature

Vlad Tsyrklevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 9 18:00:42 PDT 2018


vlad.tsyrklevich created this revision.
vlad.tsyrklevich added reviewers: pcc, eugenis.
Herald added subscribers: Sanitizers, llvm-commits, delcypher, krytarowski, emaste.

Some tests already make use of OS feature names, e.g. 'linux' and 'freebsd',
but they are not actually currently set by lit.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D49115

Files:
  test/lit.common.cfg


Index: test/lit.common.cfg
===================================================================
--- test/lit.common.cfg
+++ test/lit.common.cfg
@@ -91,8 +91,10 @@
 if platform.system() == 'Windows' and '-win' in config.target_triple:
   config.environment['LIB'] = os.environ['LIB']
 
+config.available_features.add(config.host_os.lower())
+
 if re.match(r'^x86_64.*-linux', config.target_triple):
-      config.available_features.add("x86_64-linux")
+  config.available_features.add("x86_64-linux")
 
 # Use ugly construction to explicitly prohibit "clang", "clang++" etc.
 # in RUN lines.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49115.154744.patch
Type: text/x-patch
Size: 590 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180710/a9fcebf2/attachment.bin>


More information about the llvm-commits mailing list