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

@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.

>From 734dd31948b8b0a2b8d8ec26888aaa4fd9a90e77 Mon Sep 17 00:00:00 2001
From: Icohedron <cheung.de...@gmail.com>
Date: Thu, 20 Feb 2025 18:18:26 +0000
Subject: [PATCH] Remove -emit-llvm to prevent writing

---
 clang/test/SemaHLSL/BuiltIns/and-errors.hlsl | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

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);

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to