[PATCH] D88287: [NARY-REASSOCIATE] Support reassociation of min/max
    Max Kazantsev via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Feb 25 03:04:42 PST 2021
    
    
  
mkazantsev accepted this revision.
mkazantsev added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Transforms/Scalar/NaryReassociate.cpp:315
+      (ResI = matchAndReassociateMinOrMax<umax_pred_ty>(I, OrigSCEV)) ||
+      (ResI = matchAndReassociateMinOrMax<smax_pred_ty>(I, OrigSCEV))) {
+    return ResI;
----------------
{ } not needed
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