-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[X86][test] Avoid writing to a potentially write-protected dir #108525
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This patch simply remove the -emit-llvm option as this testcase don't care about the outputed llvm IR. The current directory may be write protected e.g. in a sandboxed environment.
@phoebewang please review |
@llvm/pr-subscribers-backend-x86 @llvm/pr-subscribers-clang Author: Malay Sanghi (MalaySanghi) ChangesThis patch simply remove the -emit-llvm option as this testcase don't care about the outputed llvm IR. Full diff: https://github.com/llvm/llvm-project/pull/108525.diff 2 Files Affected:
diff --git a/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-errors.c b/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-errors.c
index c2e891217fbbcf..a5c45914cdf7d1 100644
--- a/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-errors.c
+++ b/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-errors.c
@@ -1,4 +1,4 @@
-// RUN: %clang_cc1 -flax-vector-conversions=none -ffreestanding %s -triple=i386-unknown-unknown -target-feature +avx10.2-512 -emit-llvm -Wall -Werror -verify
+// RUN: %clang_cc1 -flax-vector-conversions=none -ffreestanding %s -triple=i386-unknown-unknown -target-feature +avx10.2-512 -Wall -Werror -verify
#include <immintrin.h>
#include <stddef.h>
@@ -49,4 +49,4 @@ __m512i test_mm512_mask_cvtts_roundps_epu32(__m512i W, __mmask8 U, __m512 A) {
__m512i test_mm512_maskz_cvtts_roundps_epu32(__mmask8 U, __m512 A) {
return _mm512_maskz_cvtts_roundps_epu32(U, A, 22); // expected-error {{invalid rounding argument}}
-}
\ No newline at end of file
+}
diff --git a/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-x64-error.c b/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-x64-error.c
index 2900256914570c..334edfb501e2f0 100755
--- a/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-x64-error.c
+++ b/clang/test/CodeGen/X86/avx10_2_512satcvtds-builtins-x64-error.c
@@ -1,4 +1,4 @@
-// RUN: %clang_cc1 -flax-vector-conversions=none -ffreestanding %s -triple=x86_64-unknown-unknown -target-feature +avx10.2-512 -emit-llvm -Wall -Werror -verify
+// RUN: %clang_cc1 -flax-vector-conversions=none -ffreestanding %s -triple=x86_64-unknown-unknown -target-feature +avx10.2-512 -Wall -Werror -verify
#include <immintrin.h>
#include <stddef.h>
|
there is also |
thank you for the fix @MalaySanghi ! |
I have submitted it as a41bb71 (with 3 files). Sorry for the race, but I want it working :) |
Apologies for missing that file and thanks for taking this up. |
This patch simply remove the -emit-llvm option as this testcase don't care about the outputed llvm IR.
The current directory may be write protected e.g. in a sandboxed environment.