[PATCH] D154101: [MC] Add three-state parseDirective as a replacement for ParseDirective

[Sergei Barannikov via Phabricator via llvm-commits]

barannikov88 added inline comments.


================
Comment at: llvm/include/llvm/MC/MCParser/MCTargetAsmParser.h:149
+  constexpr bool isNoMatch() const { return Status == StatusTy::NoMatch; }
+};
+
----------------
Does anyone have a preference for ParseResult over ParseStatus? I'm not sure which one is better.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154101