[PATCH] D16227: Make context-sensitive isDereferenceable queries in isSafeToLoadUnconditionally

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 09:12:20 PST 2016


reames added a comment.

LGTM w/minor comment addressed.

(p.s. For those following, Artur pointed out to me that the use was in the original review I asked to be split into several parts.  Given that, I'm withdrawing my request for an example use.)


================
Comment at: include/llvm/Analysis/Loads.h:29
@@ -28,3 +28,3 @@
 /// specified pointer, we do a quick local scan of the basic block containing
 /// ScanFrom, to determine if the address is already accessed.
 bool isSafeToLoadUnconditionally(Value *V, unsigned Align,
----------------
reames wrote:
> Comment needs updated.
Comment still needs updated.


http://reviews.llvm.org/D16227





More information about the llvm-commits mailing list