llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang 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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
