DavidSpickett added inline comments.
================ Comment at: clang/lib/Basic/Targets/AArch64.cpp:284 + if (HasMatMul) + Builder.defineMacro("__ARM_FEATURE_MATMUL_INT8", "1"); + ---------------- I don't see specific tests for this #define ================ Comment at: clang/lib/Basic/Targets/AArch64.h:39 bool HasTME; + unsigned HasMatMul; ---------------- Why is this feature a number for AArch64, does >1 mean something? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77540/new/ https://reviews.llvm.org/D77540 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits