[LLVMdev] Purpose of -debug-compile in NVPTX backend

[Justin Holewinski]

Yes, this is a bit redundant.  It can be removed for now.


On Tue, Nov 27, 2012 at 5:04 PM, Kai <kai at redstar.de> wrote:

> Hi!
>
> My general understanding of the debug symbol support is that:
>
> - the backend defines in the XXXMCAsmInfo class if debug information is
> supported (e.g. SupportsDebugInformation = true / false)
> - if I don't want to output debug information then I can use the (hidden)
> command line option -disable-debug-info-print (defined in DwarfDebug.cpp).
>
> However, in NVPTXMCAsmInfo.cpp I found:
>
> - a new command line option:
>
> bool CompileForDebugging;
> static cl::opt<bool, true>
> Debug("debug-compile", cl::desc("Compile for debugging"), cl::Hidden,
>       cl::location(**CompileForDebugging),
>       cl::init(false));
>
> - and the assignment
>
>   SupportsDebugInformation = CompileForDebugging;
>
> Isn't this redundant code? If yes then the attached patch should be
> committed. If not then I like to know whats going on here....
>
> Regards
> Kai
>
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
>
>


-- 

Thanks,

Justin Holewinski
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20121129/2440e772/attachment.html>