[clang] [llvm] [AMDGPU] Split OPERAND_REG_IMM_INT64 into signed and unsigned variants (PR #186575)
Jay Foad via cfe-commits
cfe-commits at lists.llvm.org
Sat Mar 14 08:37:51 PDT 2026
jayfoad wrote:
> Split OPERAND_REG_IMM_INT64 into two operand types to properly handle signed vs unsigned 64-bit immediate encoding:
>
> * OPERAND_REG_IMM_INT64: Signed 64-bit integer operand
> (uses isInt<32>)
> * OPERAND_REG_IMM_B64: Unsigned 64-bit integer operand
> (uses isUInt<32>)
This is just bikeshedding but I would _much_ prefer calling them `OPERAND_REG_IMM_I64` and `OPERAND_REG_IMM_U64` to highlight the signed/unsigned difference and to match the I64 and U64 suffixes on instruction names.
https://github.com/llvm/llvm-project/pull/186575
More information about the cfe-commits
mailing list