[PATCH] D145540: [InstCombine] Reduce absolute diff from min+max+sub

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp:2413
+      match(Op1, m_OneUse(m_Intrinsic<Intrinsic::smin>(m_Specific(X),
+                                                       m_Specific(Y))))) {
+    if (I.hasNoUnsignedWrap() || I.hasNoSignedWrap()) {
----------------
RKSimon wrote:
> What about cases where we have smin(Y,X) instead?
Also, don't we have m_SMax / m_SMin ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145540