[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
Sat Sep 25 19:44:08 PDT 2021
junparser added a comment.
In D106056#3021866 <https://reviews.llvm.org/D106056#3021866>, @xbolva00 wrote:
> In D106056#3021783 <https://reviews.llvm.org/D106056#3021783>, @alexfh wrote:
>
>> FYI, we see a ~5x increase in compile time on some of our code (mainly on some large protobuf-generated files) after rG8ba2adcf9e54b34ba8efa73ac0d81a1192e4f614 <https://reviews.llvm.org/rG8ba2adcf9e54b34ba8efa73ac0d81a1192e4f614>. We're working on an isolated test case.
>
> With switches with the huge number of cases?
The iteration of switch cases does not change, so maybe It's caused by LVI->getConstantRange(SI->getCondition(), SI); ?
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