[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