[PATCH] D51616: [ExplicitGuards][NFC] API for explicit guards recognition

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 5 03:18:31 PDT 2018


mkazantsev added inline comments.


================
Comment at: include/llvm/Analysis/GuardUtils.h:25
+/// that under assumption that we enter the basic block \p BB.
+bool alwaysGoesToReExecutableDeoptBlock(const BasicBlock *BB);
+
----------------
fedor.sergeev wrote:
> Hmm... perhaps its coffee time, but I cant draw a line between the function name and its functionality.
> "Re"-executable?
Re-executable is the same as "memory was not modified before it"... I need to think how to re-state it clearly.


https://reviews.llvm.org/D51616





More information about the llvm-commits mailing list