<div dir="ltr">Ah, thanks for explaining - perhaps we could get Fred and/or Adrian to take a look at this, then?<br><br>Ideally we could split out the improvement (addressing the FIXME in the code you quoted) reviewed and committed separately - then we'd have the generic functionality you need for the unit testing API, and we could review the unit testing API separately after that. It would be unfortunate if the refactoring your doing for your use case didn't address the existing use case/need already identified here - so it seems nice to motivate it by that existing usage first.<br><br>You can probably just prototype (ensuring it passes the exsiting llvm-dsymutil tests, etc) that in code review & Fred can take a quick look ot sign off. But whatever order/way works best for you.<br><br>- Dave</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 22, 2016 at 9:47 AM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg added a comment.<br class="gmail_msg">
<br class="gmail_msg">
We actually don't need to modify the DwarfLinker, but I was planning on letting Fred Riss know that he can remove the code that works around a DIEEntry reference with DW_FORM_ref_addr after this was in. I didn't want to change the dwarf linker further to remove his work around without talking to him.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26879" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26879</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>