[llvm-commits] [patch] Add an option for not using dwarf's .loc directive

[dalej]

On Nov 18, 2010, at 5:35 PM, Rafael Espíndola wrote:

>> If you make a choice between clang vs. llvm-gcc then we will trade llvm-gcc-powerpc-darwin9 buildbot failures with llvm-gcc-x86-darwin10-gdb-testsuite build bot failures! Obviously, I care more about darwin10-gdb-testsuite results.
>> 
>> I suggest, for now while you make progress, you can entirely disable debug info for powerpc by updating following line in PPCMCAsmInfo.cpp
>>         SupportsDebugInformation= true; // Debug information.
>>  This will make all buildbots green (I do not know of any powerpc buildbot focusing on debug info).
> 
> I have attached a new version of the patches. This time no patch is
> necessary for clang.
> 
> This still uses the llvm-gcc/clang distinction. I am just finishing a
> llvm-gcc bootstrap and will run the gdb testsuite with and without
> this patch.
> 
> So, the only affected targets are ppc or all darwin 9 systems?

All darwin9.

> In any
> case, all that is needed to get llvm-gcc on some system to use .loc is
> to change:
> 
> TheTarget->setMCUseLoc(false);
> 
> with
> 
> TheTarget->setMCUseLoc(LLLVM_USE_DWARF_LOC);
> 
> and define that macro in the gcc/config directory.
> 
> Sounds like a reasonable way to do it?
> 
>> -
>> Devang
> 
> Cheers,
> Rafael
> <llvm-gcc.patch><llvm.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits