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

[Alexander Kornienko via Phabricator via llvm-commits]

alexfh added a comment.

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

> @alexfh  Turns out it was caused by EarlyTailDuplicatePass, would you try with  -mllvm -disable-early-taildup=true?

`-mllvm -disable-early-taildup=true` reduces the compilation time significantly:

  $ time ./clang-after -O3  -c -xc++ history/test.cc.reduced -o after2.o
  
  real    0m10.626s
  user    0m10.480s
  sys     0m0.146s
  
  $ time ./clang-after -O3 -mllvm -disable-early-taildup=true -c -xc++ history/test.cc.reduced -o after.o
  
  real    0m0.340s
  user    0m0.262s
  sys     0m0.077s

What's your plan here? It looks like the issue in EarlyTailDuplicatePass should be fixed before re-landing this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106056