<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 7 February 2014 10:08, James Molloy <span dir="ltr"><<a href="mailto:james.molloy@arm.com" target="_blank">james.molloy@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-GB" link="blue" vlink="purple"><p class="MsoNormal"><span style="color:rgb(31,73,125);font-family:Calibri,sans-serif;font-size:11pt">The first line did shadow just R3 but now shadows all registers. I suspect that actually this was a bug, which you have now fixed incidentally? Your patch description only mentions coprocessor registers so can you just confirm this is deliberate?</span><br>
</p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The second line has changed behaviour – previously in the i32 case no registers were shadowed – now R0-R3 are shadowed. Why is this correct? (not saying it’s not, but some justification would be good).</span></p>
</div></blockquote><div></div></div><br></div><div class="gmail_extra">I assumed both cases were the same "fix" that weren't mentioned in the commit message (and that's why I was unsure of the "fix").</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">If that's correct, I agree would be good to have a mention in the commit message. ;)</div><div class="gmail_extra"><br></div><div class="gmail_extra">cheers,</div>
<div class="gmail_extra">--renato</div></div>