<div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 23, 2012 at 11:09 AM, Sirish Pande <span dir="ltr"><<a href="mailto:spande@codeaurora.org" target="_blank">spande@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">
    There were only two reviews:<br>
    <br>
    1. Eric C asked for test cases, and that test cases be associated
    with the patch.<br>
    2. Tom Stellard had requested to split the vliw packetizer patch.<br>
    <br>
    Both have been done.<br></div></blockquote><div><br></div><div>But the review isn't finished yet. You need to wait for the reviewers to actually sign off on the change before submitting. The two comments you got were what people needed changed based on their first look, that doesn't mean the code is ready to be submitted yet.</div>
<div><br></div><div>This patch is particularly problematic because it is not just changing the Hexagon, it's changing a core part of the codegen layer. Those changes need proper review before being committed, and that hasn't happened yet.</div>
<div><br></div><div>I've reverted this patch until the review finishes because the patches requiring it had to be reverted anyways.</div><div><br></div><div>I reverted the hexagon patches because your own regression tests failed for all of the patches in the series.</div>
<div><br></div><div>This gets the tree back into both a reviewed and test-passing state. Let's keep it that way. =]</div></div></div>