[PATCH] D128648: [Clang][AArch64][SME] Add vector read/write (mova) intrinsics

[Sander de Smalen via Phabricator via cfe-commits]

sdesmalen added inline comments.


================
Comment at: clang/test/CodeGen/aarch64-sme-intrinsics/acle_sme_read.c:13
+#else
+#define ARM_STREAMING_ATTR __attribute__((arm_streaming))
+#endif
----------------
The spelling has recently changed to the `__arm_streaming`. Also with the new attribute keywords, the position of the attributes is more strict and need sto be after the function arguments (e.g. `svint8_t test_svread_..(...) ARM_STREAMING_ATTR {`)

Sorry if I previously gave you the wrong steer here to add these macros, but I've changed my mind and think it's better to remove them for now. That means we won't have any streaming attributes on the functions in the tests, but we can (and will need to) add those when we add diagnostics for missing attributes, for example when using a `shared ZA` intrinsic  when the function misses `__arm_shared_za/__arm_new_za`, or when using a streaming intrinsic when the function is not `__arm_streaming`. Writing this also made me realise the functions below would be missing `__arm_shared_za` attributes.

Can you remove these macros from the patches? Again, my apologies for the wrong steer!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128648/new/

https://reviews.llvm.org/D128648