[clang] [Clang][AArch64] Expose compatible SVE intrinsics with only +sme (PR #95787)

Paul Walker via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 18 05:17:55 PDT 2024


================
@@ -1781,7 +1781,12 @@ void SVEEmitter::createStreamingAttrs(raw_ostream &OS, ACLEKind Kind) {
   uint64_t VerifyRuntimeMode = getEnumValueForFlag("VerifyRuntimeMode");
   uint64_t IsStreamingCompatibleFlag =
       getEnumValueForFlag("IsStreamingCompatible");
+
   for (auto &Def : Defs) {
+    assert((((Def->getGuard().contains("sve") +
+              Def->getGuard().contains("sme")) <= 1) ||
+            Def->isFlagSet(VerifyRuntimeMode)) &&
----------------
paulwalker-arm wrote:

Given this is a build time error that represents a bug in `arm_sve.td`, do you think it's worth using `llvm_unreachable` rather than an assert?

https://github.com/llvm/llvm-project/pull/95787


More information about the cfe-commits mailing list