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

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