[PATCH] D136746: [mlir] Saturation arithmetic intrinsics

[Tobias Gysi via Phabricator via llvm-commits]

gysit added inline comments.


================
Comment at: mlir/include/mlir/Dialect/LLVMIR/LLVMIntrinsicOps.td:179
+def LLVM_SAddSat : LLVM_BinarySameArgsIntrOpI<"sadd.sat">;
+def LLVM_SAddSat : LLVM_BinarySameArgsIntrOpI<"uadd.sat">;
+def LLVM_SAddSat : LLVM_BinarySameArgsIntrOpI<"ssub.sat">;
----------------
Ah I missed that one. The Op names need to differ of course LLVM_SAddSat, LLVM_UAddSat, etc.


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

https://reviews.llvm.org/D136746