[PATCH] D106056: [CVP] processSwitch: Remove default case when switch cover all possible values.
JunMa via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 6 06:23:02 PDT 2021
junparser updated this revision to Diff 364774.
junparser added a comment.
@spatel, thanks for review the patch, and sorry for confusing you about the testcase.
It turns out that the last change did something wrong to get the range of switch cases.
This patch fixes it and now the testcase shows what it would handle.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106056/new/
https://reviews.llvm.org/D106056
Files:
llvm/include/llvm/Transforms/Utils/Local.h
llvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
llvm/lib/Transforms/Utils/Local.cpp
llvm/lib/Transforms/Utils/SimplifyCFG.cpp
llvm/test/Transforms/CorrelatedValuePropagation/basic.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106056.364774.patch
Type: text/x-patch
Size: 6806 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210806/263cc89f/attachment.bin>
More information about the llvm-commits
mailing list