[PATCH] Print the argument names in declarations while debugging

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 27 15:40:38 PDT 2015


Vedant Kumar <vsk at apple.com> writes:
> It'd be nice to have this, the patch LGTM.

Thanks. r248692.

> vedant
>
> On Sep 14, 2015, at 11:45 AM, Justin Bogner via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>
>> When llvm declarations have argument names, it would be nice to actually
>> print those names when debugging. Arguably, it'd be nice to print them
>> all the time, but that would mean the IR we output wouldn't round trip
>> through bitcode, which doesn't store the names.
>> 
>> The attached patch makes the varous print() methods in AsmWriter
>> optionally print "for debug" and makes the dump() methods set that
>> flag. The only thing this does differently for now is print the argument
>> names in declarations.
>> 
>> Does this seem reasonable?
>> 
>> <function-decl-debug.patch>_______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list