[PATCH] D136746: [mlir] Saturation arithmetic intrinsics

omri123 added a comment. Thanks! Submitting. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136746/new/ https://reviews.llvm.org/D136746 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[PATCH] D136746: [mlir] Saturation arithmetic intrinsics

omri123 updated this revision to Diff 470780. omri123 added a comment. Thanks again! In this revision I fixed both issues mentioned above (removed `#0` from tests and renamed the tblgen definitions) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136746/new/ https://reviews.llvm.org/D136

[PATCH] D136746: [mlir] Saturation arithmetic intrinsics

omri123 updated this revision to Diff 470775. omri123 added a comment. Thanks for your comments! In this revision I fixed the diff and updated the intrinsics to use `LLVM_BinarySameArgsIntrOpI`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136746/new/ https://reviews.llvm.org/D136746

[PATCH] D136746: [mlir] Saturation arithmetic intrinsics

omri123 created this revision. omri123 added reviewers: ftynse, gysit, llvm-commits, cfe-commits. Herald added subscribers: zero9178, bzcheeseman, awarzynski, sdasgup3, wenzhicui, wrengr, cota, teijeong, rdzhabarov, tatianashp, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, mg