[clang] [test] Remove `-emit-llvm` from the `and-errors.hlsl` test to avoid writing to a potentially write-protected directory (PR #128047)

via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 20 10:40:20 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-hlsl

Author: Deric Cheung (Icohedron)

<details>
<summary>Changes</summary>

@<!-- -->mikaelholmen [mentioned](https://github.com/llvm/llvm-project/pull/127098#discussion_r1962897888) that the `-emit-llvm` argument isn't necessary for the `and-errors.hlsl` test and may cause issues due to writing to the current (potentially write-protected) directory.

This PR removes the `-emit-llvm` argument from clang in the RUN lines of the test.

---
Full diff: https://github.com/llvm/llvm-project/pull/128047.diff


1 Files Affected:

- (modified) clang/test/SemaHLSL/BuiltIns/and-errors.hlsl (+1-2) 


``````````diff
diff --git a/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl b/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
index 0a99feb023d73..d9721140b9bf9 100644
--- a/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
+++ b/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
@@ -1,6 +1,5 @@
 // RUN: %clang_cc1 -finclude-default-header -triple \
-// RUN:   dxil-pc-shadermodel6.3-library %s \
-// RUN:   -emit-llvm -O1 -verify
+// RUN:   dxil-pc-shadermodel6.3-library %s -O1 -verify
 
 bool test_too_few_arg(bool a) {
   return __builtin_hlsl_and(a);

``````````

</details>


https://github.com/llvm/llvm-project/pull/128047


More information about the cfe-commits mailing list