[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