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

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 24 15:32:24 PDT 2021


xbolva00 added a comment.

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?


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