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

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 16 09:49:01 PDT 2017


hiraditya added a comment.

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

> Okay, so to close this out, what would we like to do about isGuaranteedToTransferExecutionToSuccessor?
>  Continue have it return false?
>
> Past that, it's clear additional bugs should be filed against gvnhoist.


I have posted the patch to fix gvnhoist here: https://reviews.llvm.org/D31035


Repository:
  rL LLVM

https://reviews.llvm.org/D30667





More information about the llvm-commits mailing list