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

Tue Aug 24 04:20:19 PDT 2021

lebedev.ri added a comment.

In D106056#2961540 <https://reviews.llvm.org/D106056#2961540>, @junparser wrote:

> 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?

Assuming that the kernel source code isn't still being miscompiled, this sounds like a bug in objtool to me?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106056