[clang] e6171e8 - [Clang][Arm] Fix fp16 return error tests under AArch64/Arm. NFC

David Green via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 29 04:16:21 PDT 2022


Author: David Green
Date: 2022-09-29T12:16:13+01:00
New Revision: e6171e87e16c59df5fba642d1f48de6f0a2d93d8

URL: https://github.com/llvm/llvm-project/commit/e6171e87e16c59df5fba642d1f48de6f0a2d93d8
DIFF: https://github.com/llvm/llvm-project/commit/e6171e87e16c59df5fba642d1f48de6f0a2d93d8.diff

LOG: [Clang][Arm] Fix fp16 return error tests under AArch64/Arm. NFC

The -fallow-half-arguments-and-returns option was removed in
59528e4bdb27ed4ab3, replaced with an always-on target option under
AArch64/Arm. There are two tests - fp16-sema.c and renderscripts.rs that
test that an error is produced for __fp16 function args/returns, which
are now expected to pass for Arm/AArch64. i.e they no longer give the
same error as before on native Arm/AArch64 machines. Alter the targets
of those tests to compensate.

Added: 
    

Modified: 
    clang/test/Sema/fp16-sema.c
    clang/test/Sema/renderscript.rs

Removed: 
    


################################################################################
diff  --git a/clang/test/Sema/fp16-sema.c b/clang/test/Sema/fp16-sema.c
index e403ba69e826..f5406c519159 100644
--- a/clang/test/Sema/fp16-sema.c
+++ b/clang/test/Sema/fp16-sema.c
@@ -1,4 +1,5 @@
-// RUN: %clang_cc1 -fsyntax-only -Wno-strict-prototypes -verify %s
+// RUN: %clang_cc1 -fsyntax-only -triple x86_64-pc-linux-gnu -Wno-strict-prototypes -verify %s
+// // REQUIRES: x86-registered-target
 
 // Functions cannot have parameters of type __fp16.
 extern void f (__fp16); // expected-error {{parameters cannot have __fp16 type; did you forget * ?}}

diff  --git a/clang/test/Sema/renderscript.rs b/clang/test/Sema/renderscript.rs
index 79b50e981411..8fa0d4389e7f 100644
--- a/clang/test/Sema/renderscript.rs
+++ b/clang/test/Sema/renderscript.rs
@@ -1,5 +1,6 @@
 // RUN: %clang_cc1 -fsyntax-only -verify -x renderscript -D__RENDERSCRIPT__ %s
-// RUN: %clang_cc1 -fsyntax-only -verify -x c %s
+// RUN: %clang_cc1 -triple x86_64-pc-linux-gnu -fsyntax-only -verify -x c %s
+// REQUIRES: x86-registered-target
 
 #ifndef __RENDERSCRIPT__
 // expected-warning at +2 {{'kernel' attribute ignored}}


        


More information about the cfe-commits mailing list