[PATCH] D124503: [InstCombine] sub(add(X,Y),umin(Y,Z)) --> add(X,usub.sat(Y,Z))

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp:2038
+    for (bool Swap : {false, true}) {
+      if (!isa<MinMaxIntrinsic>(Op1))
+        break;
----------------
I meant - can you incorporate these tests under the existing dyn_cast<MinMaxIntrinsic> check at Line 2021?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124503