[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