[PATCH] D82716: [DAGCombiner] reassociate reciprocal sqrt expression to eliminate FP division

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:13242
+          // Estimate creation failed. Clean up speculatively created nodes.
+          if (AAZ->use_empty())
+            DAG.RemoveDeadNode(AAZ.getNode());
----------------
Can we just call recursivelyDeleteUnusedNodes(AAZ) if AAZ is unused and avoid the AA handling?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82716/new/

https://reviews.llvm.org/D82716