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

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 22 12:28:57 PDT 2018


reames added inline comments.


================
Comment at: lib/Transforms/Scalar/LICM.cpp:523
 
       using namespace PatternMatch;
       if (match(&I, m_Intrinsic<Intrinsic::experimental_guard>()) &&
----------------
We should be able to sink this inside the normal hoisting now since the must execute case is now handled.   Suggestion as a follow on change, nothing more.


https://reviews.llvm.org/D50888





More information about the llvm-commits mailing list