[llvm-commits] [llvm] r98430 - /llvm/trunk/lib/Target/ARM/Thumb2SizeReduction.cpp

Bob Wilson bob.wilson at apple.com
Sat Mar 13 13:19:57 PST 2010


Yes, that looks correct to me.  Thanks, Benjamin.

On Mar 12, 2010, at 11:50 PM, Benjamin Kramer wrote:

> Author: d0k
> Date: Sat Mar 13 01:50:22 2010
> New Revision: 98430
>
> URL: http://llvm.org/viewvc/llvm-project?rev=98430&view=rev
> Log:
> Fix another warning. There is a functionality change but I believe  
> it's correct.
>
> Modified:
>    llvm/trunk/lib/Target/ARM/Thumb2SizeReduction.cpp
>
> Modified: llvm/trunk/lib/Target/ARM/Thumb2SizeReduction.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/Thumb2SizeReduction.cpp?rev=98430&r1=98429&r2=98430&view=diff
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/lib/Target/ARM/Thumb2SizeReduction.cpp (original)
> +++ llvm/trunk/lib/Target/ARM/Thumb2SizeReduction.cpp Sat Mar 13  
> 01:50:22 2010
> @@ -334,8 +334,8 @@
>     unsigned BaseReg = MI->getOperand(1).getReg();
>     ARM_AM::AMSubMode Mode = ARM_AM::getAM4SubMode(MI- 
> >getOperand(2).getImm());
>     if (BaseReg == ARM::SP &&
> -        (Entry.WideOpc == ARM::t2LDM_UPD && Mode == ARM_AM::ia) ||
> -        (Entry.WideOpc == ARM::t2STM_UPD && Mode == ARM_AM::db)) {
> +        ((Entry.WideOpc == ARM::t2LDM_UPD && Mode == ARM_AM::ia) ||
> +         (Entry.WideOpc == ARM::t2STM_UPD && Mode == ARM_AM::db))) {
>       Opc = Entry.NarrowOpc2; // tPOP or tPUSH
>       OpNum = 3;
>     } else if (!isARMLowRegister(BaseReg) || Mode != ARM_AM::ia) {
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list