[PATCH] D92183: [DAG] Legalize umin(x,y) -> sub(x,usubsat(x,y)) and umax(x,y) -> add(x,usubsat(y,x)) iff usubsat is legal

[Dave Green via Phabricator via llvm-commits]

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

LGTM. Thanks for updating the costs too.



================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:227
+      return LT.first * 2;
+    LLVM_FALLTHROUGH
+  }
----------------
LLVM_FALLTHROUGH seems to usually (perhaps always?) be written with a semicolon on the end, even if it needn't.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92183