<div>+   // For VFP load, the immediate offset is multiplied by 4.</div><div>+   unsigned Reloc =  ((TID.TSFlags & ARMII::FormMask) == ARMII::VFPLdStFrm)</div><div>+    ? ARM::reloc_arm_vfp_cp_entry : ARM::reloc_arm_cp_entry;</div>

<div><br></div><div>There's two spaces after the first = sign. Please remove one.</div><div><br></div><div>I'm a little worried that this is cut-and-pasted from getMachineOpValue(). Feel free to fix the extra space there too, by the way. Is there a sensible way to factor this logic out?</div>

<div><br></div><div><div>+     // Materialize contant pool index address.</div></div><div><br></div><div>Typo: "contant".</div><div><br></div><div>This is similar enough in implementation to emitLEApcrelInstruction that I think it's fine to go ahead and commit it as is, then make any changes Evan or others ask for afterwards.</div>

<div><br></div><div>Nick</div><br><div class="gmail_quote">On 26 May 2010 03:19, Shih-wei Liao <span dir="ltr"><<a href="mailto:sliao@google.com">sliao@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

See <a href="http://llvm.org/bugs/show_bug.cgi?id=7227" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=7227</a>: ARM JIT misses the<br>
implementation of ARM::LEApcrel.<br>
<br>
The patch passes both "make check-lit" and "make check".<br>
<br>
For "make check-lit",<br>
  Expected Passes    : 4101<br>
  Expected Failures  : 24<br>
  Unsupported Tests  : 1184<br>
make[1]: Leaving directory `/usr/local/google/upstream/llvm-obj/test'<br>
<br>
For "make check",<br>
                ===  Summary ===<br>
<br>
# of expected passes            3842<br>
# of expected failures          24<br>
make[1]: Leaving directory `/usr/local/google/upstream/llvm-obj/test'<br>
<br>
Basically, the patch added the case "ARM::LEApcrel" next to the "case<br>
ARM::LEApcrelJT". The latter was implemented, but not the former. With<br>
the added implementation, all our input programs result in correct<br>
execution under LLVM's ARM JIT.<br>
--<br>
Thanks,<br>
<font color="#888888">  Shih-wei<br>
</font><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>
<br></blockquote></div><br>