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

Ivan Kosarev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 2 01:54:45 PDT 2023


kosarev added inline comments.


================
Comment at: llvm/lib/Target/BPF/AsmParser/BPFAsmParser.cpp:520
+ParseStatus BPFAsmParser::parseDirective(AsmToken DirectiveID) {
+  return ParseStatus::NoMatch;
+}
----------------
Can we just remove this function? +Same for LoongArch, M68k, Xtensa.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154101



More information about the llvm-commits mailing list