[llvm] [ARM][TableGen][MC] Change the ARM mnemonic operands to be optional for ASM parsing (PR #83436)

Alfie Richards via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 5 05:34:41 PST 2024


================
@@ -1474,7 +1513,7 @@ void AsmMatcherInfo::buildOperandMatchInfo() {
         OperandMask |= maskTrailingOnes<unsigned>(NumOptionalOps + 1)
                        << (i - NumOptionalOps);
       }
-      if (Op.Class->IsOptional)
+      if (Op.Class->IsOptional && Op.Class->OptionalShouldOffsetCustomParsers)
----------------
AlfieRichardsArm wrote:

@kosarev I have resolved this. I had misdiagnosed an issue where certain custom parsers where behaving poorly when being applied to non-matching operands. The new patch also makes VPT operand optional and fixes these cases, removing the need for this option.

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


More information about the llvm-commits mailing list