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

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: llvm/test/Transforms/CorrelatedValuePropagation/basic.ll:380
 
+define i32 @switch_range(i32 %cond) {
+; CHECK-LABEL: @switch_range(
----------------
It would help if you pre-commit this test with the baseline CHECK lines, so it is clear what changed with this patch.
I just tried this example locally without the patch, and it does not seem to be different?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106056