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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> Staring at other places it is used, it looks like it would also think it okay to propagate various non-nullness attributes, etc past unreachable.

Propagate them to where?  unreachable doesn't have any successors.  This only makes sense if you connect the unreachable to the function's exit, or something like that.


Repository:
  rL LLVM

https://reviews.llvm.org/D30667