[PATCH] D36581: [DAG] Fix Node Replacement in PromoteIntBinOp

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

This is why I hate DAGCombine; you have to constantly worry about the CSE map invalidating pointers you care about, and I'm never confident I'm handling it correctly.

Could we simplify this code using makeEquivalentMemoryOrdering to insert the loads into the right spot in the DAG?  Or does that introduce its own RAUW problems?


https://reviews.llvm.org/D36581