[Mlir-commits] [mlir] [mlir][arith] doc updates for ub semantics, and int representations (PR #72932)
Mehdi Amini
llvmlistbot at llvm.org
Tue Nov 21 23:00:19 PST 2023
================
@@ -198,7 +198,9 @@ def Arith_AddIOp : Arith_TotalIntBinaryOp<"addi", [Commutative]> {
The `addi` operation takes two operands and returns one result, each of
these is required to be the same type. This type may be an integer scalar
type, a vector whose element type is integer, or a tensor of integers. It
- has no standard attributes.
+ has no standard attributes. If an overflow occurs, the result is the
+ mathematical value of the addition modulo 2^n, where `n` is the width of
+ the integer type.
----------------
joker-eph wrote:
And I'm not saying some LLVM style "nuw" "nsw" wouldn't be reasonable, but this isn't "the state of the world" right now.
(actually I had a proposal to just use LLVM for arithmetic, but we're not there yet)
https://github.com/llvm/llvm-project/pull/72932
More information about the Mlir-commits
mailing list