[PATCH] D26391: [JumpThreading] Unfold selects that depend on the same condition
Sebastian Pop via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 8 06:25:34 PST 2016
sebpop accepted this revision.
sebpop added a comment.
This revision is now accepted and ready to land.
LGTM.
================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:2053
+ std::sort(Selects.begin(), Selects.end());
+ auto NewEnd = std::unique(Selects.begin(), Selects.end());
+
----------------
Note to other reviewers: the above two lines "sort | uniq" are the only change from the previous patch D25477. The explanation of why this is needed is in the first comment from Pablo.
https://reviews.llvm.org/D26391
More information about the llvm-commits
mailing list