================
@@ -1,12 +1,20 @@
 // NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
 // RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sve2p1 -S 
-O1 -Werror -Wall -emit-llvm -o - %s | FileCheck %s
 // RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sve2p1 -S 
-O1 -Werror -Wall -emit-llvm -o - -x c++ %s | FileCheck %s 
-check-prefix=CPP-CHECK
+// RUN: %clang_cc1 -DSVE_OVERLOADED_FORMS -triple aarch64-none-linux-gnu 
-target-feature +sve2p1 -S -O1 -Werror -Wall -emit-llvm -o - %s | FileCheck %s
+// RUN: %clang_cc1 -DSVE_OVERLOADED_FORMS -triple aarch64-none-linux-gnu 
-target-feature +sve2p1 -S -O1 -Werror -Wall -emit-llvm -o - -x c++ %s | 
FileCheck %s -check-prefix=CPP-CHECK
 // RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sve2p1 -S 
-disable-O0-optnone -Werror -Wall -o /dev/null %s
----------------
kmclaughlin-arm wrote:

Thank you :)

I agree that having a single line to test both sve2p1 and sme2 would be 
preferable, however I'm not sure if this will be enough coverage when we add 
the streaming mode attribute to the builtins?

For SME2, we'll need to add `__arm_streaming` to the functions here as the 
tests should fail without it, but for SVE2p1 we do not need the attribute. 
@dtemirbulatov recently committed a patch for the pext & psel builtins which 
has both tests for SME2 & SVE2p1 and passes an extra option for the +sme2 lines 
to set the attribute: https://github.com/llvm/llvm-project/pull/72827.

https://github.com/llvm/llvm-project/pull/75200
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to