[PATCH] D16232: Use isSafeToLoadUnconditionally in isSafeToSpeculativelyExecute

[Artur Pilipenko via llvm-commits]

apilipenko added a comment.

isSafeToLoadUnconditionally is more aggressive than isDereferenceableAndAlignedPointer. Besides calling isDereferenceable it scans the context BB to see if the location was loaded/stored before. So, with this change isSafeToSpeculativelyExecute callers will also benefit from BB scan. Although, so far in practice the only place where isSafeToSpeculativelyExecute is called with a context is LICM.


http://reviews.llvm.org/D16232