[PATCH] Clarify the pass name of X86AsmPrinter

Eli Bendersky eliben at google.com
Fri Feb 15 16:41:51 PST 2013


Hello,

Currently the x86 AsmPrinter reports its name as a hardcoded "X86
AT&T-Style Assembly Printer". This is somewhat misleading in printouts
(such as -time-passes and others) when object-file emission is used
(X86AsmPrinter emits either an object file or assembly via a specific
MCStreamer).

This patch replaces this string (and a related comment) with something
a bit more generic. A more audacious approach would be for the method
to check which streamer it has and return a precise name. Note also
that this isn't a x86-specific problem - x86 is just being used as an
example here. A similar change can be applied to other targets.

PTAL,
Eli
-------------- next part --------------
A non-text attachment was scrubbed...
Name: asmprinter-name.1.patch
Type: application/octet-stream
Size: 1067 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130215/b68191aa/attachment.obj>


More information about the llvm-commits mailing list