[PATCH] D137549: [AsmParser] Match mandatory operands following optional operands.

Ivan Kosarev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 7 06:02:21 PST 2022


kosarev added a comment.

I think this could also make the hack added in https://reviews.llvm.org/D41598 unnecessary but it doesn't because of https://reviews.llvm.org/D20527 not respecting types of optional parameters, meaning mandatory parameters that follow optional ones may be off their positions and fail to match, which is a problem of its own.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137549



More information about the llvm-commits mailing list