[PATCH] D34278: [InstCombine] Recognize and simplify three way comparison idioms

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 21:30:59 PDT 2017


mkazantsev accepted this revision.
mkazantsev added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: lib/Transforms/InstCombine/InstCombineCompares.cpp:2441
+                                           ConstantInt *&Greater) {
+
+  // TODO: Generalize this to work with other comparison idioms or ensure
----------------
Why do we need this blank line?


================
Comment at: lib/Transforms/InstCombine/InstCombineCompares.cpp:2507
+  return nullptr;
+
+}
----------------
Same here.


https://reviews.llvm.org/D34278





More information about the llvm-commits mailing list