[PATCH] D82905: [AIX] Add system-aix to config file

[Shuhong Liu via Phabricator via llvm-commits]

ShuhongL created this revision.
ShuhongL added a reviewer: daltenty.
ShuhongL added a project: LLVM.
Herald added subscribers: llvm-commits, delcypher.

This is a complementary patch to D82100 <https://reviews.llvm.org/D82100> since the aix builbot is still running the unsupported test shtest-format-argv0. Add system-aix to the sub llvm-lit config.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D82905

Files:
  llvm/utils/lit/lit/llvm/config.py
  llvm/utils/lit/tests/lit.cfg
  llvm/utils/lit/tests/shtest-format-argv0.py


Index: llvm/utils/lit/tests/shtest-format-argv0.py
===================================================================
--- llvm/utils/lit/tests/shtest-format-argv0.py
+++ llvm/utils/lit/tests/shtest-format-argv0.py
@@ -5,7 +5,7 @@
 #
 # This test is not supported on AIX since `[` is only available as a shell builtin
 # and is not installed under PATH by default.
-# UNSUPPORTED: aix
+# UNSUPPORTED: system-aix
 #
 # RUN: %{lit} -j 1 -v %{inputs}/shtest-format-argv0 | FileCheck %s
 
Index: llvm/utils/lit/tests/lit.cfg
===================================================================
--- llvm/utils/lit/tests/lit.cfg
+++ llvm/utils/lit/tests/lit.cfg
@@ -87,7 +87,7 @@
   if sys.platform.startswith('win') or sys.platform.startswith('cygwin'):
     config.available_features.add('system-windows')
   if platform.system() == 'AIX':
-    config.available_features.add('aix')
+    config.available_features.add('system-aix')
 
 # For each of lit's internal shell commands ('env', 'cd', 'diff', etc.), put
 # a fake command that always fails at the start of PATH.  This helps us check
Index: llvm/utils/lit/lit/llvm/config.py
===================================================================
--- llvm/utils/lit/lit/llvm/config.py
+++ llvm/utils/lit/lit/llvm/config.py
@@ -51,12 +51,14 @@
         elif platform.system() == 'Windows':
             # For tests that require Windows to run.
             features.add('system-windows')
-        elif platform.system() == "Linux":
+        elif platform.system() == 'Linux':
             features.add('system-linux')
         elif platform.system() in ['FreeBSD']:
             features.add('system-freebsd')
-        elif platform.system() == "NetBSD":
+        elif platform.system() == 'NetBSD':
             features.add('system-netbsd')
+        elif platform.system() == 'AIX':
+            features.add('system-aix')
 
         # Native compilation: host arch == default triple arch
         # Both of these values should probably be in every site config (e.g. as


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82905.274583.patch
Type: text/x-patch
Size: 2018 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200630/98020fc2/attachment.bin>