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

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 27 01:49:47 PDT 2018


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





More information about the llvm-commits mailing list