[PATCH] D11852: [LowerSwitch] Fix a bug when LowerSwitch deletes the default block

Tue Aug 11 08:40:06 PDT 2015

hans added a comment.

In http://reviews.llvm.org/D11852#221347, @chenli wrote:

> In http://reviews.llvm.org/D11852#221321, @reames wrote:
>
> > The revised patch is strictly less powerful than the current code and the previous patch.  If a block occurs earlier in the iteration order than the only block which reaches it and we then delete the reaching edge, we will now fail to remove the unreachable block.  I would prefer the first patch posted over the revised one.
>
>
> Yes, you are correct. I didn't think about this case. I will submit the initial patch if Hans is ok with that.

Sure.

http://reviews.llvm.org/D11852