[PATCH] D50501: [LICM] Hoist guards with invariant conditions

[Max Kazantsev via Phabricator via llvm-commits]

mkazantsev planned changes to this revision.
mkazantsev added inline comments.


================
Comment at: lib/Transforms/Scalar/LICM.cpp:461
+    // Keep track of whether the prefix instructions could have written memory.
+    bool IsMemoryNotModified = true;
 
----------------
reames wrote:
> Bug: Can only start true if a) readonly loop or b) header block.
Indeed. Thanks for pointing out!


https://reviews.llvm.org/D50501