[PATCH] D136352: [AArch64] Add SVE2.1 target feature for Armv9-A 2022 Architecture Extension

[David Sherwood via Phabricator via llvm-commits]

david-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.td:159
                 "sve2 or sme">;
+def HasSVE2p1orSME
+    : Predicate<"Subtarget->hasSVE2p1() || Subtarget->hasSME()">,
----------------
paulwalker-arm wrote:
> I know the precedent has been set but I find this awkward to parse.  Is `HasSVE2p1_or_HasSME` a terrible idea? If agreeable we can always convert the other instance later on.
I don't mind changing it, but I do think it makes sense to also fix up the other names too for consistency (perhaps in a later patch?). I guess your reason for wanting to put 'Has' twice in the name is to make it easily searchable?

@sdesmalen any thoughts?


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

https://reviews.llvm.org/D136352