Hi Tim,<br><br>  Thanks for your reply.  I have checked the code again.  You are right.<br>fixup_t2_condbranch and fixup_t2_uncondbranch is always PC-relative.<br>Thus, I'm removing the duplicated code in the new patch set.  Please<br>
have a look.  Thanks.<br><br>Sincerely,<br>Logan<br><br><br><div class="gmail_quote">On Wed, Aug 29, 2012 at 5:59 PM, 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 Logan,<br>
<div class="im"><br>
>   It seems that ARMELFObjectWriter is not translating fixup_t2_condbranch<br>
> and fixup_t2_uncondbranch into R_ARM_THM_JUMP24.  As a result, "llc"<br>
> can not compile following code correctly with integrated assembler.  I hope<br>
> these patches can fix this issue.  Please review.  Thanks!<br>
<br>
</div>The results look good to me, but I'm a little confused about the<br>
duplication for both IsPCRel and !IsPCRel. Shouldn't these fixups<br>
always be PC-relative?<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim.<br>
</font></span></blockquote></div><br>