[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