[PATCH] D147299: [InstCombine] Add support for maximum(a,b) + minimum(a,b) => a + b

[Serguei Katkov via Phabricator via llvm-commits]

skatkov added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp:1811
+                     m_c_Intrinsic<Intrinsic::minimum>(m_Deferred(X),
+                                                       m_Deferred(Y))))) {
+    BinaryOperator *Result = BinaryOperator::CreateFAddFMF(X, Y, &I);
----------------
mkazantsev wrote:
> We could also match `min(a, b) + max(b, a)`
we do, see the test @test_comm2.
Actually it was m_c versus m_ does.


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

https://reviews.llvm.org/D147299