<div dir="ltr">I prefer the first patch because the purpose of this patch is all for rare debug output and nothing else. In the first patch everything is nicely compartmented in a file scope function, and it does not touch anything else. In the second patch, you did plumbing all the way from the Linking Context to the debug routine, so many member functions now take one more extra argument. I think it doesn't worth.</div>

<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 24, 2014 at 9:37 AM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM.<div class="HOEnZb"><div class="h5"><br>
<br>
On 2/24/2014 2:15 AM, Nico Rieck wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On <a href="tel:24.02.2014%2006" value="+12402201406" target="_blank">24.02.2014 06</a>:43, Shankar Easwaran wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This patch makes the strings duplicated in the Registry(Reader.cpp,<br>
referenceKindToString) and the LayoutPass. I am not sure if there is a<br>
way to use the Registry here, but I think it would be better to have a<br>
function that could call the Registry to get this information.<br>
</blockquote>
It's possible by passing down a Registry reference to the LayoutPass<br>
ctor in {Core,ELF,PECOFF,Macho}<u></u>LinkingContext::addPasses(). The<br>
LayoutPass would then require ReaderWriter/Reader.h. Would this be<br>
acceptable? Updated patch attached.<br>
<br>
-Nico<br>
</blockquote>
<br>
<br></div></div><div class="HOEnZb"><div class="h5">
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>