[PATCH] D106056: [CVP] processSwitch: Remove default case when switch cover all possible values.

[JunMa via Phabricator via llvm-commits]

junparser added a comment.

In D106056#2961201 <https://reviews.llvm.org/D106056#2961201>, @efriedma wrote:

> The backend can, in general, create basic blocks that don't contain any instructions, and don't fall through to another block.  A jump table entry can refer to such a block.  I guess certain tools could be confused by this.
>
> If that's the issue, it should be possible to work around it using "-mllvm -trap-unreachable".

Thanks for the help! I wonder if we can enable this by default after this patch? Any suggest?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106056/new/

https://reviews.llvm.org/D106056