<div style="font-family: arial, helvetica, sans-serif"><font size="2">Thanks for the thorough review Tim! Committed r158722.<br><br><div class="gmail_quote">On Tue, Jun 19, 2012 at 1:40 AM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jan,<br>
<div class="im"><br>
>> The condition code didn't actually matter for arm "b" instructions,<br>
>> unlike "bl".  It should just use the R_ARM_JUMP24 reloc.<br>
><br>
</div><div class="im">> Ok to commit?  Patch reattached, based on r158659<br>
<br>
</div>It looks correct to me. The ARM ELF ABI says that a linker is<br>
permitted to convert anything relocated with R_ARM_CALL to a "BLX"<br>
which would be flat-out wrong for a "B". It also says that R_ARM_PLT32<br>
is deprecated in favour of R_ARM_JUMP24 (etc) so the PCRel changes<br>
should be correct. Tested too.<br>
<br>
You can commit as far as I'm concerned (whatever that counts for).<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim.<br>
</font></span></blockquote></div><br></font></div>