[llvm-commits] [PATCH] To handle s* registers in emitVFPLoadStoreMultipleInstruction()

Zonr Chang zonr.xchg at gmail.com
Tue May 25 04:00:05 PDT 2010


I reviewed and tested this patch. Thanks for your hard working on reporting
and resolving the bug in ARM JIT.
LGTM.

Zonr

From: Shih-wei Liao <sliao at google.com>
> Date: Tue, May 25, 2010 at 1:21 AM
> Subject: [PATCH] To handle s* registers in
> emitVFPLoadStoreMultipleInstruction()
> To: llvm-commits <llvm-commits at cs.uiuc.edu>
>
>
> Could someone review the patch in
> http://llvm.org/bugs/show_bug.cgi?id=7221?
>
> --- lib/Target/ARM/ARMCodeEmitter.cpp
> +++ lib/Target/ARM/ARMCodeEmitter.cpp
>       break;
>     ++NumRegs;
>   }
> -  Binary |= NumRegs * 2;
> +  // bit 8 will be set if <list> is consecutive 64-bit registers (e.g.,
> d0)
> +  if(Binary & 0x100)
> +    Binary |= NumRegs * 2;
> +  else
> +    Binary |= NumRegs;
>
>   emitWordLE(Binary);
>  }
>
> Thanks.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20100525/21615a4a/attachment.html>


More information about the llvm-commits mailing list