[PATCH] D123515: [RISCV] Support '.option arch' directive

luxufan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 12 09:29:08 PDT 2022


StephenFan added inline comments.


================
Comment at: llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp:2200
+    Parser.Lex();
+    return Parser.parseToken(AsmToken::EndOfStatement,
+                             "unexpected token, expected end of statement");
----------------
jrtc27 wrote:
> binutils allows more than one change in the same line as `.option arch, +foo, bar, -baz, +qwerty` etc
what would `.option arch, bar` do?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123515



More information about the llvm-commits mailing list