[all-commits] [llvm/llvm-project] 5cebb0: [mlir][intrange] Represent bounds of `ReflectBound...

[Felix Schneider via All-commits]

  Branch: refs/heads/users/ubfx/testops-umax-umin-unsigned-printing
  Home:   https://github.com/llvm/llvm-project
  Commit: 5cebb095abb336a34fffbb430494599815eeea1b
      https://github.com/llvm/llvm-project/commit/5cebb095abb336a34fffbb430494599815eeea1b
  Author: Felix Schneider <fx.schn at gmail.com>
  Date:   2024-05-18 (Sat, 18 May 2024)

  Changed paths:
    M mlir/test/Dialect/Arith/int-range-interface.mlir
    M mlir/test/Dialect/Arith/int-range-opts.mlir
    M mlir/test/lib/Dialect/Test/TestOpDefs.cpp

  Log Message:
  -----------
  [mlir][intrange] Represent bounds of `ReflectBoundsOp` as `si`/`ui`

This patch adapts the `test.reflect_bounds` test Op to use explicitly
signed and unsigned representation for signed and unsigned bounds of
`IntegerType`s.

This is mostly a cosmetic change as the internal representation of the
ranges is unchanged. However, it improves readability of tests.

Example:
```mlir
// old:
test.reflect_bounds {smax = 127 : i8, smin = -128 : i8, umax = -56 : i8, umin = 100 : i8}
// new:
test.reflect_bounds {smax = 127 : si8, smin = -128 : si8, umax = 200 : ui8, umin = 100 : ui8}
```



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications