[llvm] [lit][aix] Always use internal lit shell on AIX (PR #113355)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 22 11:16:58 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-testing-tools
Author: Daniel Paoliello (dpaoliello)
<details>
<summary>Changes</summary>
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.
---
Full diff: https://github.com/llvm/llvm-project/pull/113355.diff
1 Files Affected:
- (modified) llvm/utils/lit/lit/llvm/config.py (+7)
``````````diff
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")
``````````
</details>
https://github.com/llvm/llvm-project/pull/113355
More information about the llvm-commits
mailing list