[llvm] r201723 - Make one statement easier to understand from post commmit feedback from a
Reed Kotler
rkotler at mips.com
Wed Feb 19 15:02:16 PST 2014
Should say "can be satisfied by MipsAsmPrinter"
On 02/19/2014 02:59 PM, reed kotler wrote:
> Here is a proposed patch then:
>
> rkotler at mipssw006:~/llvm_trunk$ svn diff
> Index: lib/Target/Mips/Mips16ISelLowering.cpp
> ===================================================================
> --- lib/Target/Mips/Mips16ISelLowering.cpp (revision 201723)
> +++ lib/Target/Mips/Mips16ISelLowering.cpp (working copy)
> @@ -467,8 +467,11 @@
> // So for now we always save S2. The optimization will be done
> // in a follow-on patch.
> //
> - if (1 || (Signature->RetSig != Mips16HardFloatInfo::NoFPRet))
> - FuncInfo->setSaveS2();
> + // FIXME: add this conditional when optimization to not save S2
> + // can be satisfied by AsmPrinter.
> + //
> + // if (Signature->RetSig != Mips16HardFloatInfo::NoFPRet)
> + FuncInfo->setSaveS2();
> }
> // one more look at list of intrinsics
> if (std::binary_search(Mips16IntrinsicHelper,
>
>
> On 02/19/2014 02:35 PM, Eric Christopher wrote:
>> 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-8NJIiSa5LzA at public.gmane.org> 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-Tmj1lob9twqVc3sceRu5cw at public.gmane.org
>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list