[all-commits] [llvm/llvm-project] be96bd: [mlir][emitc] Don't emit extra semicolon after bra...

[Kirill Chibisov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: be96bd74f8eae5637033e4e05fcbf2a693ce8334
      https://github.com/llvm/llvm-project/commit/be96bd74f8eae5637033e4e05fcbf2a693ce8334
  Author: Kirill Chibisov <contact at kchibisov.com>
  Date:   2025-01-14 (Tue, 14 Jan 2025)

  Changed paths:
    M mlir/lib/Target/Cpp/TranslateToCpp.cpp
    M mlir/test/Target/Cpp/declare_func.mlir
    M mlir/test/Target/Cpp/for.mlir
    M mlir/test/Target/Cpp/if.mlir
    A mlir/test/Target/Cpp/no_extra_semicolon.mlir
    M mlir/test/Target/Cpp/switch.mlir

  Log Message:
  -----------
  [mlir][emitc] Don't emit extra semicolon after bracket (#122464)

Extra semicolons were emitted for operations that should never have
them, since not every place was checking whether semicolon would be
actually needed.

Thus change the emitOperation to ignore trailingSemicolon field for such
operations.



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