[llvm-commits] [llvm] r172630 - in /llvm/trunk: include/llvm/MC/MCContext.h lib/MC/MCDwarf.cpp test/MC/MachO/gen-dwarf-producer.s tools/llvm-mc/llvm-mc.cpp

[Eric Christopher]

Hi Kevin,

Awesome, please do. And thanks for coming up with an interim way to test :)

Thanks!

-eric


On Wed, Jan 16, 2013 at 9:55 AM, Kevin Enderby <enderby at apple.com> wrote:

> Hi Eric,
>
> This is just for testing (without the clang change).  I didn't want to add
> a it as a command line argument to llvm-mc as that would then have the
> producer string as it would also affect the AT_Apple flags.
>
> If you like I can remove this code and the test when the clang side of the
> change is finished.
>
> Kev
>
> On Jan 16, 2013, at 9:52 AM, Eric Christopher <echristo at gmail.com> wrote:
>
>
>
> +static std::string DwarfDebugProducer;
>> +static void setDwarfDebugProducer(void) {
>> +  if(!getenv("DEBUG_PRODUCER"))
>> +    return;
>> +  DwarfDebugProducer += getenv("DEBUG_PRODUCER");
>> +}
>> +
>>
>
> Any particular reason or compatibility with this? I'd prefer not to have
> it otherwise. If we do need it can you make it work for the compiled case
> as well?
>
> Thanks!
>
> -eric
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130116/62b1a729/attachment.html>