1. Fixed in r104652.<div><br></div><div>2. Passed "make check":</div><div><div><span class="Apple-tab-span" style="white-space:pre">              </span>===  Summary ===</div><div><br></div><div>   # of expected passes<span class="Apple-tab-span" style="white-space:pre">             </span>3841</div>
<div>   # of expected failures<span class="Apple-tab-span" style="white-space:pre">             </span>24</div><div>   make[1]: Leaving directory `/usr/local/google/upstream/llvm-obj/test'</div><div><br></div><div>3. Passed "make check-lit":</div>
<div><div>   Expected Passes    : 4100</div><div>   Expected Failures  : 24</div><div>   Unsupported Tests  : 1184</div></div><div><div>   make[1]: Leaving directory `/usr/local/google/upstream/llvm-obj/test'</div></div>
<div><br></div><div>Thanks for the reviews. I'm learning the patching from you --- I called it arm-jit-*.patch to follow your style.</div><br><div class="gmail_quote">On Tue, May 25, 2010 at 4:00 AM, Zonr Chang <span dir="ltr"><<a href="mailto:zonr.xchg@gmail.com">zonr.xchg@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I reviewed and tested this patch. Thanks for your hard working on reporting and resolving the bug in ARM JIT.<div>LGTM.<div>
<br></div><font color="#888888"><div>Zonr</div></font><div><div></div><div class="h5"><div><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


From: Shih-wei Liao <<a href="mailto:sliao@google.com" target="_blank">sliao@google.com</a>><br>
Date: Tue, May 25, 2010 at 1:21 AM<br>
Subject: [PATCH] To handle s* registers in emitVFPLoadStoreMultipleInstruction()<br>
To: llvm-commits <<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a>><br>
<br>
<br>
Could someone review the patch in <a href="http://llvm.org/bugs/show_bug.cgi?id=7221" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=7221</a>?<br>
<br>
--- lib/Target/ARM/ARMCodeEmitter.cpp<br>
+++ lib/Target/ARM/ARMCodeEmitter.cpp<br>
       break;<br>
     ++NumRegs;<br>
   }<br>
-  Binary |= NumRegs * 2;<br>
+  // bit 8 will be set if <list> is consecutive 64-bit registers (e.g., d0)<br>
+  if(Binary & 0x100)<br>
+    Binary |= NumRegs * 2;<br>
+  else<br>
+    Binary |= NumRegs;<br>
<br>
   emitWordLE(Binary);<br>
 }<br>
<br>
Thanks.<font color="#888888"><font color="#000000"><br></font></font></blockquote></div><br></div></div></div></div>
</blockquote></div><br><br clear="all"><br>-- <br>Thanks,<br>  Shih-wei<br>
</div>