[PATCH] D10920: Unify isSafeToLoadUnconditionally and isDereferenceablePointer

[hfinkel@anl.gov via llvm-commits]

hfinkel added inline comments.

================
Comment at: lib/Analysis/Loads.cpp:72
@@ -69,2 +71,3 @@
+                                       const TargetLibraryInfo *TLI) {
   // Zero alignment means that the load has the ABI alignment for the target
   if (Align == 0)
----------------
Why can this part not be unified?

It seems this is the only difference. Why don't we just eliminate this function in favor of isDereferenceableAndAlignedPointer?



http://reviews.llvm.org/D10920