[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