[PATCH] D53783: [Intrinsic] Signed and Unsigned Saturation Subtraction Intirnsics

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added a comment.

nit: there is a spelling error in the title "Intirnsics"



================
Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:4964
+    // SatMax -> Overflow && SumDiff < 0
+    // SatMin -> Overflow && SumDiff > 0
     APInt MinVal = APInt::getSignedMinValue(BitWidth);
----------------
It should say ">= 0" here.


Repository:
  rL LLVM

https://reviews.llvm.org/D53783