[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