[PATCH] D30667: GVNHoist: handle basic blocks with UnreachableInst

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 07:56:17 PST 2017


hiraditya added a comment.

In https://reviews.llvm.org/D30667#693739, @dberlin wrote:

> Uh, won't this already return false on the call to longjmp?


GVNHoist only calls isGuaranteedToTransferExecutionToSuccessor for terminators.


https://reviews.llvm.org/D30667





More information about the llvm-commits mailing list