[all-commits] [llvm/llvm-project] e50372: [TableGen] Emit integers in GlobalISelMatchTable a...

[Piotr Fusik via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e50372c14c074242414ff878c2f2b1fd507ba177
      https://github.com/llvm/llvm-project/commit/e50372c14c074242414ff878c2f2b1fd507ba177
  Author: Piotr Fusik <p.fusik at samsung.com>
  Date:   2025-08-13 (Wed, 13 Aug 2025)

  Changed paths:
    M llvm/test/TableGen/GlobalISelCombinerEmitter/match-table-typeof.td
    M llvm/test/TableGen/GlobalISelCombinerEmitter/match-table.td
    M llvm/test/TableGen/GlobalISelEmitter/GlobalISelEmitter.td
    A llvm/test/TableGen/GlobalISelEmitter/int64min.td
    M llvm/utils/TableGen/Common/GlobalISel/GlobalISelMatchTable.cpp

  Log Message:
  -----------
  [TableGen] Emit integers in GlobalISelMatchTable as unsigned (#153391)

The numbers are stored in an `uint8_t` array, either directly
or via `GIMT_Encode2/4/8` macros
(see `getEncodedEmitStr` and `emitEncodingMacrosDef`).
Passing negative numbers only adds confusion and results in a warning
for the `INT64_MIN` value (C++ parses `-9223372036854775808` as two
tokens: a unary minus and an unsigned integer, triggered in #151687).

BTW, parenthesize parameters in `GIMT_Encode2/4/8` macros
and avoid unnecessary casts.



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