[PATCH] fix llvm linker issue with dbg.declare

Manman Ren mren at apple.com
Wed Jan 30 09:44:36 PST 2013


Hi all,

Since I got no response on this patch and we need this patch soon, I committed at r173946.
If you have any concern about this patch, please review after commit.

Thanks,
Manman

On Jan 29, 2013, at 2:01 PM, Manman Ren <mren at apple.com> wrote:

> 
> ping
> 
> Thanks,
> Manman
> 
> On Jan 28, 2013, at 3:21 PM, Manman Ren <mren at apple.com> wrote:
> 
>> 
>> Hello,
>> 
>> Given source IR:
>> call void @llvm.dbg.declare(metadata !{i32* %argc.addr}, metadata !14), !dbg !15
>> we used to generate 
>> call void @llvm.dbg.declare(metadata !27, metadata !28), !dbg !29
>> !27 = metadata !{null}
>> 
>> With this patch, we will generate
>> call void @llvm.dbg.declare(metadata !{i32* %argc.addr}, metadata !27), !dbg !28
>> 
>> Looking up %argc.addr in ValueMap will return null. Since %argc.addr is already
>> correctly set up, we can use identity mapping.
>> 
>> Please review,
>> 
>> Thanks,
>> Manman
>> 
>> <linker.patch>_______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 
> _______________________________________________
> 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