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