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

JunMa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 24 18:21:00 PDT 2021


junparser added a comment.

In D106056#2963343 <https://reviews.llvm.org/D106056#2963343>, @nathanchance wrote:

> Thank you for all of the insight so far, I can confirm that adding `-mllvm -trap-unreachable` to `KBUILD_{C,LD}FLAGS` does resolve the issue. I have brought up this discussion with the `objtool` maintainers here <https://lore.kernel.org/r/5913cdf4-9c8e-38f8-8914-d3b8a3565d73@kernel.org/> and I will report back here depending on what they say; hopefully you do not mind holding off on merging the patch until there is a response.

LGTM, Let's hold this patch and wait your update.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106056



More information about the llvm-commits mailing list