<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 15, 2017 at 7:47 AM George Rimar <<a href="mailto:grimar@accesssoftek.com">grimar@accesssoftek.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif">
<p>><span style="color:rgb(33,33,33);font-size:12pt">Not quite sure what you mean by "on linker side" - but I guess you mean using linker features like comdats etc, rather than DWARF parsing/reassembly/etc.</span></p>
<p><span style="color:rgb(33,33,33);font-size:12pt"><br>
</span></p>
</div><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><p><span style="color:rgb(33,33,33);font-size:12pt">I mean that it probably not a good idea for external library. I feel it is much more convinent to do such proccessing in a linker.</span></p></div></blockquote><div>Ah, a matter of where the logic is implemented.<br><br>My concern is that we already have one use of this logic (dsymutil) and at least two other places that could benefit from it (lld and dwp) - it would seem unfortunate to build and maintain 3 separate versions of this.<br><br>(also, possibly one day, a 4th place: llvm IR linking - it could be useful to generate type DWARF (& CodeView) from frontends rather than backends, then do a full DWARF aware linking of it in the llvm linking step)<br><br>But all of that's probably a bit of a long way off I imagine, in terms of getting the initial idea (should lld grow/have DWARF aware linking functionality) going.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif">
<p>Linker do and knows much more about things like sections that are ICFed, eliminated, about COMDATs and many things like strings dedups.<br>
</p>
<p>So it can rebuild relocations and perform DWARF deduplication probably in a faster/more convinent way probably than external API could provide.<br>
</p>
<p>Honestly I did not yet think about it too much, just current feelings.<br></p></div></blockquote><div>Fair, fair.<br><br>- Dave </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><p>
</p>
<p><br>
</p>
<p>Geоrge.<br>
</p>
</div>

</blockquote></div></div>