================
@@ -15,7 +17,7 @@
 // CPP-CHECK-NEXT:    [[TMP0:%.*]] = tail call target("aarch64.svcount") 
@llvm.aarch64.sve.ptrue.c8()
 // CPP-CHECK-NEXT:    ret target("aarch64.svcount") [[TMP0]]
 //
-svcount_t test_svptrue_c8(void) {
+svcount_t test_svptrue_c8(void) __arm_streaming_compatible {
----------------
sdesmalen-arm wrote:

Can you rewrite the tests in such a way that we do test for the two different 
configurations:
* For SVE2p1: no attribute
* For SME2: `__arm_streaming`

You can do something with macros, so that you replace 
`__arm_streaming_compatible` here with `ATTR` and then do something like:
```
#ifndef TEST_SME
#define ATTR
#else
#define ATTR __arm_streaming
#endif
```
and then for the `+sme2` RUN line you'd add `-DTEST_SME`.

That way, we're sure to test for the correct mode when we update the builtin to 
use a different attribute than `__arm_streaming_compatible`.

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

Reply via email to