Hi all,<div><br></div><div>As Renato's suggestion, I split the last patch into two patches. 'fast-isel-fastcc.patch' is for fix, and another is for clean up.</div><div><br></div><div>I will commit them separately if approval, please review them, thanks.</div>
<div><br></div><div>Jush<br><br><div class="gmail_quote">On Fri, Aug 10, 2012 at 6:21 PM, Renato Golin <span dir="ltr"><<a href="mailto:rengolin@systemcall.org" target="_blank">rengolin@systemcall.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 10 August 2012 04:49, Jush Lu <<a href="mailto:jush.msn@gmail.com">jush.msn@gmail.com</a>> wrote:<br>

> This part doesn’t fix anything, it just simplifies if statements, no<br>
> functionality changes in this part.<br>
<br>
</div>I think this is an even bigger reason to split the patch.<br>
<br>
The fix should be minimalistic and have all possible tests associated,<br>
to make it easier to roll-back, further analyse and to know what that<br>
patch was intended to fix in the first place, say, an year from now.<br>
<br>
It doesn't matter much if you want to submit the fix first or the<br>
clean up first. Clean-ups also generally don't need additional tests.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
cheers,<br>
--renato<br>
<br>
<a href="http://systemcall.org/" target="_blank">http://systemcall.org/</a><br>
</div></div></blockquote></div><br></div>