[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