[PATCH] D53053: [Intrinsic] Signed Saturation Addition Intrinsic

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: lib/CodeGen/TargetLoweringBase.cpp:1895
+  SDValue SumPos = DAG.getSetCC(dl, BoolVT, Sum, Zero, ISD::SETGE);
+  SDValue SumNeg = DAG.getSetCC(dl, BoolVT, Sum, Zero, ISD::SETLT);
+  SDValue UseSatMax = DAG.getNode(ISD::AND, dl, BoolVT, Overflow, SumNeg);
----------------
What if we did the selects like this? Then we don't need any ANDs and we only need one setcc.

overflow ? (sum < 0 ? satmax : satmin) : sum


Repository:
  rL LLVM

https://reviews.llvm.org/D53053