<div dir="ltr">Hi Stepan,<div><br></div><div style>You are right, the test remained broken until I cleaned the directory. However, after Jakob's enquires, I think we should still keep it off until another round of review is done. Shouldn't be a problem, since the patch is localized and doesn't depend on APIs that change too often.</div>
<div style><br></div><div style>If Jacob is happy with your patch, I'm fine with it as long as it passes at least check-all and test-suite on an A8 or higher and on x86_64.</div><div style><br></div><div style>cheers,</div>
<div style>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 7 April 2013 12:51, Stepan Dyatkovskiy <span dir="ltr"><<a href="mailto:stpworld@narod.ru" target="_blank">stpworld@narod.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
Eh.. guys, I looked at clang-native-arm-lnt after my patch was reverted,<br>
Revert of r178854 didn't fix any of failed tests, revision number of revert commit is 178883.  In #452 build (r178904) it is still failed (few hours after Renato reverted my commits). Actually it have been fixed somewhere between 178904 and 178923).<br>

<br>
But I'm still very sorry about ugly r178851 and *absent* reaction after 178854. Currently my beagleboard rebuilds everything, so I'll publish test reports soon.<br>
<br>
I also have question about TargetRegisterInfo::<u></u>getMinimalPhysRegClass and QPRRegClass.contains(). My purpose in this code is to find the minimal class for register. I supposed this method does exactly what I need.<br>

So do you propose to replace it with SPRRegClass/DPRRegClass/<u></u>QPRRegClass.contains methods set?<span class="HOEnZb"><font color="#888888"><br>
<br>
-Stepan.</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
Jakob Stoklund Olesen wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Apr 5, 2013, at 11:36 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 5 April 2013 19:26, Jakob Stoklund Olesen <<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>> wrote:<br>
<br>
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>
<br>
Hi Jacob,<br>
<br>
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.<br>
<br>
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.<br>

</blockquote>
<br>
Thanks, Renato.<br>
<br>
I agree.<br>
<br>
/jakob<br>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div>