[llvm] 4583dbc - [lit][aix] Always use internal lit shell on AIX (#113355)

via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 23 12:18:06 PDT 2024


Author: Daniel Paoliello
Date: 2024-10-23T12:18:02-07:00
New Revision: 4583dbc7b85d25bdda652ad1afe5ec3e95183a16

URL: https://github.com/llvm/llvm-project/commit/4583dbc7b85d25bdda652ad1afe5ec3e95183a16
DIFF: https://github.com/llvm/llvm-project/commit/4583dbc7b85d25bdda652ad1afe5ec3e95183a16.diff

LOG: [lit][aix] Always use internal lit shell on AIX (#113355)

Diff on AIX doesn't have all the required features used in tests (see
<https://github.com/llvm/llvm-project/pull/108871> and
<https://github.com/llvm/llvm-project/pull/112997#issuecomment-2429656192>),
so always use the internal shell.

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/utils/lit/lit/llvm/config.py b/llvm/utils/lit/lit/llvm/config.py
index 5f762ec7f3514a..1ef5796cd32e44 100644
--- a/llvm/utils/lit/lit/llvm/config.py
+++ b/llvm/utils/lit/lit/llvm/config.py
@@ -57,6 +57,13 @@ def __init__(self, lit_config, config):
                 self.lit_config.note("using lit tools: {}".format(path))
                 lit_path_displayed = True
 
+        if platform.system() == "AIX":
+            # Diff on AIX doesn't have all the required features (see
+            # https://github.com/llvm/llvm-project/pull/108871 and
+            # https://github.com/llvm/llvm-project/pull/112997#issuecomment-2429656192)
+            # so always use the internal shell.
+            self.use_lit_shell = True
+
         if platform.system() == "OS/390":
             self.with_environment("_BPXK_AUTOCVT", "ON")
             self.with_environment("_TAG_REDIR_IN", "TXT")


        


More information about the llvm-commits mailing list