[all-commits] [llvm/llvm-project] 71db97: [mlir][emitc] Arith to EmitC: Handle addi, subi an...

Matthias Gehre via All-commits all-commits at lists.llvm.org
Fri Mar 22 07:40:14 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 71db97152173a524a3e16e02b7fdc50f405c8695
      https://github.com/llvm/llvm-project/commit/71db97152173a524a3e16e02b7fdc50f405c8695
  Author: Matthias Gehre <matthias.gehre at amd.com>
  Date:   2024-03-22 (Fri, 22 Mar 2024)

  Changed paths:
    M mlir/lib/Conversion/ArithToEmitC/ArithToEmitC.cpp
    A mlir/test/Conversion/ArithToEmitC/arith-to-emitc-failed.mlir
    M mlir/test/Conversion/ArithToEmitC/arith-to-emitc.mlir

  Log Message:
  -----------
  [mlir][emitc] Arith to EmitC: Handle addi, subi and muli (#86120)

Important to consider that `arith` has wrap around semantics, and in C++
signed overflow is UB.
Unless the operation guarantees that no signed overflow happens, we will
perform the arithmetic in an equivalent unsigned type.
`bool` also doesn't wrap around in C++, and is not addressed here.



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


More information about the All-commits mailing list