<div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 12, 2012 at 1:01 PM, Matthew Curtis <span dir="ltr"><<a href="mailto:mcurtis@codeaurora.org" target="_blank" class="cremed">mcurtis@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div>Any objections to submitting this
      patch?<br></div></div></blockquote><div><br></div><div>It hasn't been review yet...</div><div><br></div><div>Anyways, I'll take this as a ping.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"><div>
      <br>
      Matthew Curtis.<div class="im"><br>
      <br>
      On 9/6/2012 4:24 PM, Matthew Curtis wrote:<br>
    </div></div>
    <blockquote type="cite"><div class="im">Hello
      all,
      <br>
      <br>
      This patch updates the Hexagon ToolChain:</div></blockquote></div></blockquote><div>Rather than one patch that is a grab-bag of updates, how about posting patches incrementally addressing the issues?</div><div><br>
</div><div>You seem to already have them broken apart here:</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000"><blockquote type="cite">
<div class="im">
      - reimplement hexagon::Link::ConstructJob to call linker directly
      <br>
        rather than going through gcc
      <br>
      - add/improve support for small data threshold, pic, and other
      args
      <br>
      - add overrides for include paths, et. al.
      <br>
      - minor refactoring</div></blockquote></div></blockquote><div>If possible, can you post the minor refactoring (no functionality changes) patch first? It will likely make the others easier to review.</div><div><br></div>
<div><br></div><div>Also, there are no test updates or new test cases despite a very significant set of changes. Each patch needs to either be a simple refactoring (no functionality changes), or to have an update to a test case, or to have a new test case to cover the new behavior.</div>
</div></div>