[PATCH] D100170: [NARY] Don't optimize min/max if there are side uses

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 11 21:37:26 PDT 2021


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

LGTM



================
Comment at: llvm/test/Transforms/NaryReassociate/nary-req.ll:5
+
+declare i32 @llvm.smax.i32(i32 %a, i32 %b)
+
----------------
Please precommit the test with current output and put your patch on top of it so that it clearly shows the difference.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100170



More information about the llvm-commits mailing list