[PATCH] Debug Info: emit linkage name for constructor/destructor

Eric Christopher echristo at gmail.com
Wed May 8 14:24:13 PDT 2013


Hi Yacine,

Thanks for the patch, I've gone ahead and cleaned it up a bit and
committed it here:

        A       test/DebugInfo/X86/linkage-name-ctor-dtor.ll
        M       lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp
        M       test/DebugInfo/X86/concrete_out_of_line.ll
Committed r181471

-eric

On Sat, May 4, 2013 at 4:42 AM, Yacine Belkadi
<yacine.belkadi.1 at gmail.com> wrote:
> On 05/04/2013 12:46 AM, David Blaikie wrote:
>> Looks mostly reasonable - haven't thought about it in detail yet.
>>
>> Could you do the checks with one dump/check instead of two, though?
>
> Sure.
> I initially used two dump/checks so the test wouldn't depend on the order of
> the DWARF tags (constructor one relative to the destructor one) in the dump.
> Please find attached the updated patch (I also changed some wording in the
> commit message).
>
> Thanks,
> Yacine
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



More information about the llvm-commits mailing list