[PATCH] D60673: [SimplifyCFG] Improove and speed up ReduceSwitchRange
Shawn Landden via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 17 10:10:05 PDT 2019
shawnl marked an inline comment as done.
shawnl added a comment.
This is a combination of two patches. To see them separately: https://github.com/shawnl/llvm/commits/test
This patch now depends on D60823 <https://reviews.llvm.org/D60823>
================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:5579
+ auto *Rotr = Builder.Insert(CallInst::Create(Fshr, {Sub, Sub, ShiftC}));
+ SI->replaceUsesOfWith(SI->getCondition(), Rotr);
----------------
nikic wrote:
> Just looking at this, shouldn't there be some changes to existing tests?
There were no tests for the old behavior, surprisingly.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60673/new/
https://reviews.llvm.org/D60673
More information about the llvm-commits
mailing list