[llvm-commits] CVS: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp

Chris Lattner sabre at nondot.org
Tue Jul 19 09:56:29 PDT 2005


On Tue, 19 Jul 2005, Nate Begeman wrote:
> Remove unnecessary FP_EXTEND. This causes worse codegen for SSE.

This actually caused incorrect code to be generated, right?

-Chris

> Diffs of the changes:  (+0 -3)
>
> SelectionDAGISel.cpp |    3 ---
> 1 files changed, 3 deletions(-)
>
>
> Index: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
> diff -u llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1.65 llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1.66
> --- llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1.65	Tue Jul  5 14:57:53 2005
> +++ llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp	Tue Jul 19 11:50:03 2005
> @@ -403,9 +403,6 @@
>       Op1 = DAG.getNode(ISD::ZERO_EXTEND, TmpVT, Op1);
>     break;
>   case MVT::f32:
> -    // Extend float to double.
> -    Op1 = DAG.getNode(ISD::FP_EXTEND, MVT::f64, Op1);
> -    break;
>   case MVT::i64:
>   case MVT::f64:
>     break; // No extension needed!
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>

-Chris

-- 
http://nondot.org/sabre/
http://llvm.org/




More information about the llvm-commits mailing list