Backends support for debug_str merging

7 views
Skip to first unread message

Diana Chen

unread,
Sep 15, 2014, 5:05:41 AM9/15/14
to mcli...@googlegroups.com, Simon Atanasyan, Hemant Kulkarni, Shankar Easwaran
Hi,

We're working on debug string table merging. Two new APIs in Relocator has been added to support it.
1. Relocator::getDebugStringOffset - get the offset of an input .debug_str according to given relocation.
2. Relocator::applyDebugStringOffset - apply the output .debug_str offset to relocation target according to given relocation and offset. 

These two APIs in ARM/AArch64/Mips/Mips64/X86/X86_64 have been done. Please check the branch debug-string.


Hi Simon,
Could you please help to review the Mips part? Thank you.

Hi Shankar and Hemant,
Could you please help to add the Hexagon part? Thank you.

Also please let me know if there is any suggestion or question.

Thanks,
Diana

Simon Atanasyan

unread,
Sep 18, 2014, 6:47:33 AM9/18/14
to Diana Chen, mcli...@googlegroups.com
Hi Diana,

The MIPS part and the patch in general look good to me. I have not
found any regressions.

Simon

Pete Chou

unread,
Sep 19, 2014, 1:02:12 AM9/19/14
to Simon Atanasyan, Diana Chen, mcli...@googlegroups.com
BTW, safe ICF is also target-specific, and is disabled on MIPS. You may want to take a look at that if you want to enable the opt.

Thanks,
Pete

--
You received this message because you are subscribed to the Google Groups "MCLinker" group.
To unsubscribe from this group and stop receiving emails from it, send an email to mclinker+u...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply all
Reply to author
Forward
0 new messages