[llvm] r201723 - Make one statement easier to understand from post commmit feedback from a

[Eric Christopher]

Uh, this is just if (true)... how about just put a FIXME in there and
remove the conditional?

-eric

On Wed, Feb 19, 2014 at 2:11 PM, Reed Kotler <rkotler at mips.com> wrote:
> Author: rkotler
> Date: Wed Feb 19 16:11:45 2014
> New Revision: 201723
>
> URL: http://llvm.org/viewvc/llvm-project?rev=201723&view=rev
> Log:
> Make one statement easier to understand from post commmit feedback from a
> review of the previous patch that introduced this week.
>
>
> Modified:
>     llvm/trunk/lib/Target/Mips/Mips16ISelLowering.cpp
>
> Modified: llvm/trunk/lib/Target/Mips/Mips16ISelLowering.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/Mips16ISelLowering.cpp?rev=201723&r1=201722&r2=201723&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/Mips16ISelLowering.cpp (original)
> +++ llvm/trunk/lib/Target/Mips/Mips16ISelLowering.cpp Wed Feb 19 16:11:45 2014
> @@ -467,7 +467,7 @@ getOpndList(SmallVectorImpl<SDValue> &Op
>            // So for now we always save S2. The optimization will be done
>            // in a follow-on patch.
>            //
> -          if (Signature->RetSig != Mips16HardFloatInfo::NoFPRet || 1)
> +          if (1 || (Signature->RetSig != Mips16HardFloatInfo::NoFPRet))
>              FuncInfo->setSaveS2();
>          }
>          // one more look at list of intrinsics
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits