[PATCH] D50888: [NFC][LICM] Remove too conservative variable

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 20 00:58:34 PDT 2018


mkazantsev added inline comments.


================
Comment at: lib/Transforms/Scalar/LICM.cpp:464
+    // mem-writing operations in every block, e.g. using something similar to
+    // isGuaranteedToExecute.
     bool IsMemoryNotModified = CurLoop->getHeader() == BB;
----------------
fedor.sergeev wrote:
> Since you are going to remove isGuaranteedToExecute perhaps mention something else here?
> Or just drop the reference.
Right, I will do it when I drop it.


https://reviews.llvm.org/D50888





More information about the llvm-commits mailing list