[llvm-commits] [llvm] r168485 - /llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp

[Jakub Staszak]

Testcase?

On Nov 22, 2012, at 4:31 AM, Lang Hames <lhames at gmail.com> wrote:

> Author: lhames
> Date: Wed Nov 21 21:31:45 2012
> New Revision: 168485
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=168485&view=rev
> Log:
> llvm.fmuladd.* lowering should be checking isOperationLegalOrCustom, rather than
> isOperationLegal. Thanks to Craig Topper for pointing this out.
> 
> Modified:
>    llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
> 
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp?rev=168485&r1=168484&r2=168485&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp Wed Nov 21 21:31:45 2012
> @@ -4994,7 +4994,7 @@
>   case Intrinsic::fmuladd: {
>     EVT VT = TLI.getValueType(I.getType());
>     if (TM.Options.AllowFPOpFusion != FPOpFusion::Strict &&
> -        TLI.isOperationLegal(ISD::FMA, VT) &&
> +        TLI.isOperationLegalOrCustom(ISD::FMA, VT) &&
>         TLI.isFMAFasterThanMulAndAdd(VT)){
>       setValue(&I, DAG.getNode(ISD::FMA, dl,
>                                getValue(I.getArgOperand(0)).getValueType(),
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits