[PATCH] D39340: Modifying reassociate for improved CSE

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 27 05:53:20 PDT 2017


hiraditya added inline comments.


================
Comment at: lib/Transforms/Scalar/Reassociate.cpp:2217
+
+        unsigned MaxRank = std::max(Ops[i].Rank, Ops[j].Rank);
+        if (Score > Max || (Score == Max && MaxRank < BestRank)) {
----------------
Does this ensure that the original order of evaluation is emitted when there is no preferred order?


Repository:
  rL LLVM

https://reviews.llvm.org/D39340





More information about the llvm-commits mailing list