[llvm] r214601 - [SDAG] Let the DAG combiner take care of dead nodes rather than manually

Chandler Carruth chandlerc at gmail.com
Fri Aug 1 17:19:10 PDT 2014


Author: chandlerc
Date: Fri Aug  1 19:19:10 2014
New Revision: 214601

URL: http://llvm.org/viewvc/llvm-project?rev=214601&view=rev
Log:
[SDAG] Let the DAG combiner take care of dead nodes rather than manually
deleting them. This already seems to work, as no tests fail without
this.

Modified:
    llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp

Modified: llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp?rev=214601&r1=214600&r2=214601&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp Fri Aug  1 19:19:10 2014
@@ -8443,8 +8443,6 @@ static SDValue PerformVMOVRRDCombine(SDN
     if (DCI.DAG.getTargetLoweringInfo().isBigEndian())
       std::swap (NewLD1, NewLD2);
     SDValue Result = DCI.CombineTo(N, NewLD1, NewLD2);
-    DCI.RemoveFromWorklist(LD);
-    DAG.DeleteNode(LD);
     return Result;
   }
 





More information about the llvm-commits mailing list