[PATCH] D136746: [mlir] Saturation arithmetic intrinsics

Tobias Gysi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 26 04:26:47 PDT 2022


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



More information about the cfe-commits mailing list