[llvm] 9d88543 - [AIX] Use internal lit shell for TableGen instead of a global setting (#113627)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 25 10:06:05 PDT 2024
Author: Abhina Sree
Date: 2024-10-25T13:06:02-04:00
New Revision: 9d88543301f262e584a36ea969237a2cf054328b
URL: https://github.com/llvm/llvm-project/commit/9d88543301f262e584a36ea969237a2cf054328b
DIFF: https://github.com/llvm/llvm-project/commit/9d88543301f262e584a36ea969237a2cf054328b.diff
LOG: [AIX] Use internal lit shell for TableGen instead of a global setting (#113627)
This is to address the latest lit regressions
https://lab.llvm.org/buildbot/#/builders/64/builds/1285 caused by using
the internal lit shell. This change will limit using the internal lit
shell to TableGen on AIX so we do not hit these regressions.
Added:
Modified:
llvm/test/TableGen/lit.local.cfg
llvm/utils/lit/lit/llvm/config.py
Removed:
################################################################################
diff --git a/llvm/test/TableGen/lit.local.cfg b/llvm/test/TableGen/lit.local.cfg
index 0e827479cd4123..9d6dfdc14bbfb0 100644
--- a/llvm/test/TableGen/lit.local.cfg
+++ b/llvm/test/TableGen/lit.local.cfg
@@ -1,2 +1,10 @@
+import platform
+import lit.formats
+
config.suffixes = [".td"]
config.excludes = ["Common", "Inputs"]
+
+# AIX '
diff ' command doesn't support --strip-trailing-cr, but the internal
+# python implementation does, so use that for cross platform compatibility
+if platform.system() == "AIX":
+ config.test_format = lit.formats.ShTest()
diff --git a/llvm/utils/lit/lit/llvm/config.py b/llvm/utils/lit/lit/llvm/config.py
index 1ef5796cd32e44..5f762ec7f3514a 100644
--- a/llvm/utils/lit/lit/llvm/config.py
+++ b/llvm/utils/lit/lit/llvm/config.py
@@ -57,13 +57,6 @@ 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