[PATCH] D43936: [Asm] Refactor debug printing of AsmToken

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 6 05:37:02 PST 2018


SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

Looks like a sensible change to me.



================
Comment at: lib/MC/MCParser/MCAsmLexer.cpp:99
+  case AsmToken::Tilde:          OS << "Tilde"; break;
+  case AsmToken::PercentCall16:
+    OS << "PercentCall16";
----------------
Nit: I see this code is copy-pasted here, but I was wondering why from this
case the style is different, i.e. single line vs. multiple lines.


Repository:
  rL LLVM

https://reviews.llvm.org/D43936





More information about the llvm-commits mailing list