[llvm-commits] [PATCH] Debug info utilities and printer pass

Chris Lattner clattner at apple.com
Wed Dec 10 18:13:10 PST 2008


On Dec 10, 2008, at 1:13 PM, Devang Patel wrote:
>> 02-intrin.patch:
>> Introduce DbgFuncStart::getDisplayName() (which refers to unmangled C
>> ++
>> name!), and DbgDeclareInst::getVariableName()
>> Introduce include/llvm/Support/DebugInfo.h:
>
> We already have include/llvm/Analysis/DebugInfo.h, so I suggest that
> you use another name for this new header.

More strongly, why do we need this new code when we have llvm/Analysis/ 
Debuginfo.h?  What does this do that llvm/Analysis/DebugInfo doesn't,  
or shouldn't do?

I'd really rather make Analysis/DebugInfo.h better rather than  
introduce a new parallel way to analyze debug info.

-Chris




More information about the llvm-commits mailing list