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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 20 05:23:51 PDT 2021


lebedev.ri added a comment.

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

> In D106056#2957062 <https://reviews.llvm.org/D106056#2957062>, @lebedev.ri wrote:
>
>> Please rebased past 5d4f37e895487408e61498ec42e545ec5f778b5b <https://reviews.llvm.org/rG5d4f37e895487408e61498ec42e545ec5f778b5b> - does it help?
>
> yes, this also works for PR51531 but not the kernel boot,

Thanks for checking.

> @nathanchance,  It seems there are someting wrong with other pass(maybe after CVP), Still it is better to get reduced case.

Yep, we need an actionable reproducer.


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