<div dir="ltr">Hi Larry,<div><br></div><div>Thanks for taking a look at this. The patch looks good in general, but I'm concerned about the overhead of scanning all operands for all instructions when looking for a <span style="font-size:13px;font-family:arial,sans-serif">MO_GOT_ABSOLUTE_</span><span style="font-size:13px;font-family:arial,sans-serif">ADDRESS</span>. Is it possible to move the scan in to <span style="font-size:13px;font-family:arial,sans-serif">X86MCInstLower::Lower and only do it once we see an </span><span style="font-size:13px;font-family:arial,sans-serif">MO_</span><span style="font-size:13px;font-family:arial,sans-serif">GlobalAddress?</span></div>
<div><span style="font-size:13px;font-family:arial,sans-serif"><br></span></div><div><font face="arial, sans-serif">Cheers,</font></div><div><font face="arial, sans-serif">Lang.</font></div><div><span style="font-size:13px;font-family:arial,sans-serif"><br>
</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 2, 2014 at 12:35 PM, Larry D'Anna <span dir="ltr"><<a href="mailto:larry@elder-gods.org" target="_blank">larry@elder-gods.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I tested this by self-compiling a i686 llvm and running the regression tests:<br>
<br>
   * applied patch<br>
<br>
   * compiled llvm+clang with native compiler<br>
<br>
   * used the llvm I just built to build a i686-linux-gnu llvm<br>
<br>
   * ran the tests on that<br>
<br>
I also scanned the resulting object files for the optimization that was disabled<br>
by the FIXMEs that I removed.  It should look like a add32ri instruction with a<br>
GOTPC reloc on the immediate operand, which has been folded into a add32mi<br>
instruction.  This did occur in ARMDisassembler.cpp.<br>
<br>
Sorry for the mis-formatted attempt at submitting the patch.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>