[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/lib/Target/BPF/AsmParser/BPFAsmParser.cpp:520
+ParseStatus BPFAsmParser::parseDirective(AsmToken DirectiveID) {
+  return ParseStatus::NoMatch;
+}
----------------
kosarev wrote:
> Can we just remove this function? +Same for LoongArch, M68k, Xtensa.
I guess we can. The old method was pure virtual and had to be overridden, the new one does not have to.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154101