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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
