[Mlir-commits] [mlir] [mlir] Strip spaces from `host_cxx`/`host_cc` in tests (PR #128542)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Mon Feb 24 09:49:14 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir
Author: Ivan Butygin (Hardcode84)
<details>
<summary>Changes</summary>
Continuation of https://github.com/llvm/llvm-project/pull/128439
Some builders have spaces at the end of the `host_cc` path, which now breaks.
---
Full diff: https://github.com/llvm/llvm-project/pull/128542.diff
1 Files Affected:
- (modified) mlir/test/lit.cfg.py (+2-2)
``````````diff
diff --git a/mlir/test/lit.cfg.py b/mlir/test/lit.cfg.py
index 32b2f8b53d5fa..6222acfafdcdd 100644
--- a/mlir/test/lit.cfg.py
+++ b/mlir/test/lit.cfg.py
@@ -55,8 +55,8 @@
config.substitutions.append(("%shlibext", config.llvm_shlib_ext))
config.substitutions.append(("%llvm_src_root", config.llvm_src_root))
config.substitutions.append(("%mlir_src_root", config.mlir_src_root))
-config.substitutions.append(("%host_cxx", config.host_cxx))
-config.substitutions.append(("%host_cc", config.host_cc))
+config.substitutions.append(("%host_cxx", config.host_cxx.strip()))
+config.substitutions.append(("%host_cc", config.host_cc.strip()))
# Searches for a runtime library with the given name and returns the found path.
``````````
</details>
https://github.com/llvm/llvm-project/pull/128542
More information about the Mlir-commits
mailing list