[PATCH] D32526: [LLVM][inline-asm] Altmacro absolute expression '%' feature

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 04:07:31 PDT 2017


rengolin added a comment.

I think this patch and https://reviews.llvm.org/D32523 are small and simple enough that it could be merged into one...



================
Comment at: MC/MCParser/AsmParser.cpp:489
   bool parseDirectiveAltmacro(StringRef Directive);
+  void setAltMacroMode() {
+    getLexer().SetAltMacroMode(true);
----------------
These functions seem redundant... You could have used `getLexer()` directly.


https://reviews.llvm.org/D32526





More information about the llvm-commits mailing list