[llvm] [CVP] Check whether the default case is reachable (PR #79993)
Quentin Dian via llvm-commits
llvm-commits at lists.llvm.org
Sat Feb 10 01:01:10 PST 2024
DianQK wrote:
> > > > > I fixed the regression by changing the default case from `%default.unreachable` to another BB :(
> > > >
> > > >
> > > > An “arbitrarily meaningless” BB?
> > >
> > >
> > > Yeah :)
> >
> >
> > I have considered a similar approach in my optimization PR. I'm pretty sure this has to be the wrong approach. Since unreachable provides more facts, this will definitely help with the final machine code. Also see [#78582 (comment)](https://github.com/llvm/llvm-project/pull/78582#issuecomment-1898566450).
>
> I'm just saying that the key to this problem is the unreachable default case.
I get it. Please ignore me. :3
https://github.com/llvm/llvm-project/pull/79993
More information about the llvm-commits
mailing list