[PATCH] D28241: [LICM] Untangle some of the logic in LICM canSinkOrHoistInst

Xin Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 8 00:03:59 PST 2017


trentxintong added a comment.

@hfinkel can you please confirm this is correct. Basically, I do not think we need to check for isSafeToExecuteUnconditionally for the set of SSA instructions we handle when we are sinking.


https://reviews.llvm.org/D28241





More information about the llvm-commits mailing list