[llvm-commits] [llvm] r100315 - in /llvm/trunk/lib/Target/X86/AsmPrinter: X86AsmPrinter.h X86MCInstLower.cpp

[Chris Lattner]

On Apr 5, 2010, at 11:09 AM, Devang Patel wrote:

>>> 
>>> Ideally, there should be a target independent way to print DBG_VALUE machine instructions as comments.
>> 
>> Do we actually want this long term?  I've been regarding the comments as a hack that will go away when the Dwarf writer knows how to deal with dbg_value.
> 
> At the moment, the DwarfDebug does not remove them from the instruction stream. The nice thing about printing comment is that it is only printed when -verbose-asm is used.

I completely agree devang, they should be printed as comments (by target independent code) when verbose asm is enabled.  I pointed this out a long time ago.

-Chris