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

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev 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);
+
----------------
mkazantsev wrote:
> 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.
Do you really need it to be  a public interface?


https://reviews.llvm.org/D51616