[libc++] Improve Lit's buildhost=XXXX feature on a few platforms
authorLouis Dionne <ldionne.2@gmail.com>
Thu, 23 Jun 2022 19:09:52 +0000 (15:09 -0400)
committerLouis Dionne <ldionne.2@gmail.com>
Mon, 27 Jun 2022 12:33:44 +0000 (08:33 -0400)
Differential Revision: https://reviews.llvm.org/D128455

libcxx/test/libcxx/transitive_includes.sh.cpp
libcxx/utils/libcxx/test/features.py

index 7c4ff67..19d6ab0 100644 (file)
@@ -32,7 +32,7 @@
 // UNSUPPORTED: gcc
 
 // This test doesn't work on AIX or Windows, but it should. Needs investigation.
-// XFAIL: buildhost=aix6, buildhost=windows
+// XFAIL: buildhost=aix, buildhost=windows
 
 // Prevent <ext/hash_map> from generating deprecated warnings for this test.
 #if defined(__DEPRECATED)
index d805ae6..93842f2 100644 (file)
@@ -260,9 +260,11 @@ DEFAULT_FEATURES += [
 # The build host could differ from the target platform for cross-compilation.
 DEFAULT_FEATURES += [
   Feature(name='buildhost={}'.format(sys.platform.lower().strip())),
-  # sys.platform can be represented by "sub-system" on Windows host, such as 'win32', 'cygwin', 'mingw' & etc.
-  # Here is a consolidated feature for the build host plaform name on Windows.
-  Feature(name='buildhost=windows', when=lambda cfg: platform.system().lower().startswith('windows'))
+  # sys.platform can often be represented by a "sub-system", such as 'win32', 'cygwin', 'mingw', freebsd13 & etc.
+  # We define a consolidated feature on a few platforms.
+  Feature(name='buildhost=windows', when=lambda cfg: platform.system().lower().startswith('windows')),
+  Feature(name='buildhost=freebsd', when=lambda cfg: platform.system().lower().startswith('freebsd')),
+  Feature(name='buildhost=aix', when=lambda cfg: platform.system().lower().startswith('aix'))
 ]
 
 # Detect whether GDB is on the system, has Python scripting and supports