[PATCH] D159188: [AArch64][SME] Make the overloaded svreinterpret_* functions streaming-compatible.

Sander de Smalen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 1 08:59:25 PDT 2023


sdesmalen added inline comments.


================
Comment at: clang/include/clang/Basic/Attr.td:418
 def TargetARM : TargetArch<["arm", "thumb", "armeb", "thumbeb"]>;
-def TargetAArch64 : TargetArch<["aarch64"]>;
+def TargetAArch64 : TargetArch<["aarch64", "aarch64_be"]>;
 def TargetAnyArm : TargetArch<!listconcat(TargetARM.Arches, TargetAArch64.Arches)>;
----------------
david-arm wrote:
> I'm not sure why this change is included in this patch?
This is needed for `clang/test/Sema/aarch64-sve-intrinsics/big_endian.cpp` which  has `aarch64_be` as target and includes arm_sve.h.
Without this change the `__arm_streaming_compatible` attribute is unsupported and leads to loads of error messages, thus failing the test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159188



More information about the cfe-commits mailing list