[PATCH] D88287: [NARY-REASSOCIATE] Support reassociation of min/max

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 03:45:54 PST 2021


RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/NaryReassociate.cpp:595
+      const SCEV *RHSExpr = SE->getSCEV(RHS);
+      for (uint j = 0; j < 2; ++j) {
+        if (j == 0) {
----------------
@ebrevnov uint is an unknown type on most targets - breaking buiilds - just use int ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88287/new/

https://reviews.llvm.org/D88287



More information about the llvm-commits mailing list