<div dir="ltr">On 5 April 2013 19:26, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">That's not right either. How are you testing this code? Please don't just guess.<br>
<br>
You should probably be using QPRRegClass.contains().<br></blockquote><div></div></div><br></div><div class="gmail_extra" style>Hi Jacob,</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>I have reverted this (and the original) patch already. This is not right and there isn't enough tests to make me sleep at night.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>Unless I see clear evidence that a patch in that area is actually doing what it should (via review of someone that knows it, like you) AND proper tests are passing (including the test-suite on ARM), it'll remain reverted.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>cheers,</div><div class="gmail_extra" style>--renato</div></div>