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

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

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

> In https://reviews.llvm.org/D30667#699749, @efriedma wrote:
>
> >
>


...

> IE I was told the semantics of unreachable say that if we are guaranteed to hit the unreachable, along a given codepath, we can assume that code path cannot execute.

This is my understanding as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D30667