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

Oliver Stannard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 6 05:48:25 PST 2018


olista01 added inline comments.


================
Comment at: lib/MC/MCParser/MCAsmLexer.cpp:99
+  case AsmToken::Tilde:          OS << "Tilde"; break;
+  case AsmToken::PercentCall16:
+    OS << "PercentCall16";
----------------
SjoerdMeijer wrote:
> 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.
I assume that this was done because a lot of these are too long to keep the alignment with the ones above. I'll re-format this  in a more consistent way before committing.


Repository:
  rL LLVM

https://reviews.llvm.org/D43936





More information about the llvm-commits mailing list