[PATCH] D12833: Combines pair of rotr/rotl with constant shifts into one instruction with combined shift

Elena Demikhovsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 3 09:22:07 PDT 2017


delena added a comment.

I suggest to submit the test first to show the current code. It will let us to see the optimization.



================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:5289
+        bool NextIsRotr = NextOp == ISD::ROTR;
+        bool SameSide = MainIsRotr == NextIsRotr;
+
----------------
bool SameSide = (N->getOpcode() == NextOp);


https://reviews.llvm.org/D12833





More information about the llvm-commits mailing list