<div dir="ltr">I agree with Joerg. This patch is a good one improving code quality and is not really related to or at least does not block the suggested refactoring. We should submit this patch.</div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Fri, Sep 6, 2013 at 10:59 AM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@netbsd.org" target="_blank">joerg@netbsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
  That doesn't make sense. If anything, the argument will change from LinkerInput to FileNode. At the same time, I still don't believe that merging them is necessarily a good idea, but I don't want to focus on that part while having such a large diff pending.<br>


<br>
<a href="http://llvm-reviews.chandlerc.com/D1598" target="_blank">http://llvm-reviews.chandlerc.com/D1598</a><br>
</blockquote></div><br></div>