Fix custom target lowering for zero/any/sign_extend: make sure that
authorScott Michel <scottm@aero.org>
Wed, 30 Apr 2008 00:26:38 +0000 (00:26 +0000)
committerScott Michel <scottm@aero.org>
Wed, 30 Apr 2008 00:26:38 +0000 (00:26 +0000)
DAG.UpdateNodeOperands() is called before (not after) the call to
TLI.LowerOperation().

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@50461 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/SelectionDAG/LegalizeDAG.cpp

index f964fb6be51c1297acb0c5ebdba396163b653eb2..f8e44ec69567fdca3fca30dae54760e313647e22 100644 (file)
@@ -3874,14 +3874,12 @@ SDOperand SelectionDAGLegalize::LegalizeOp(SDOperand Op) {
     case Expand: assert(0 && "Shouldn't need to expand other operators here!");
     case Legal:
       Tmp1 = LegalizeOp(Node->getOperand(0));
+      Result = DAG.UpdateNodeOperands(Result, Tmp1);
       if (TLI.getOperationAction(Node->getOpcode(), Node->getValueType(0)) ==
           TargetLowering::Custom) {
-        Tmp2 = TLI.LowerOperation(Result, DAG);
-        if (Tmp2.Val) {
-          Tmp1 = Tmp2;
-        }
+        Tmp1 = TLI.LowerOperation(Result, DAG);
+        if (Tmp1.Val) Result = Tmp1;
       }
-      Result = DAG.UpdateNodeOperands(Result, Tmp1);
       break;
     case Promote:
       switch (Node->getOpcode()) {