[PATCH] D40184: [LICM] Fix PR35342

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: lib/Transforms/Scalar/LICM.cpp:909
     if (!DT->isReachableFromEntry(User->getParent())) {
       User->replaceUsesOfWith(&I, UndefValue::get(I.getType()));
+      UI = I.user_begin();
----------------
Could we solve the problem by instead changing this call to replaceUsesOfWith with "U = UndefValue::get(I.getType());"?  We shouldn't be resetting the iterators like this if we can avoid it.


https://reviews.llvm.org/D40184