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

Mon Sep 27 03:57:51 PDT 2021

junparser added a comment.

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

> In D106056#3023777 <https://reviews.llvm.org/D106056#3023777>, @xbolva00 wrote:
>
>> In D106056#3023709 <https://reviews.llvm.org/D106056#3023709>, @alexfh wrote:
>>
>>> We'd appreciate a prompt fix or a workaround. If neither is possible, please revert the commit while working on a proper solution.
>>
>> Well, it is not "common" code, switch 1000 cases. But easy fix is probably to introduce some reasonable limit when to bail out.
>
>
>
> In D106056#3023777 <https://reviews.llvm.org/D106056#3023777>, @xbolva00 wrote:
>
>> In D106056#3023709 <https://reviews.llvm.org/D106056#3023709>, @alexfh wrote:
>>
>>> We'd appreciate a prompt fix or a workaround. If neither is possible, please revert the commit while working on a proper solution.
>>
>> Well, it is not "common" code, switch 1000 cases. But easy fix is probably to introduce some reasonable limit when to bail out.
>
> switch 1000 cases is not the issue, use loop induction variable as condition is.  It cause LVI recursive call SolveBlockValue.

@xbolva00  I was wrong.  The compiler time regression is caused by  CodeGen Passes rather than CVP or LVI.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106056/new/

https://reviews.llvm.org/D106056