[llvm] [TableGen] Fix OperandMap table size (PR #125424)

[Rahul Joshi via llvm-commits]

jurahul wrote:

Yeah that will work. Though I am changing the code here in
https://github.com/llvm/llvm-project/pull/125313

If you want I can incorporate that change in the PR (since both names are
changing in that PR)

On Sun, Feb 2, 2025 at 12:10 PM Mariusz Sikora ***@***.***>
wrote:

> ***@***.**** commented on this pull request.
> ------------------------------
>
> In llvm/utils/TableGen/InstrInfoEmitter.cpp
> <https://github.com/llvm/llvm-project/pull/125424#discussion_r1938569560>:
>
> > @@ -300,8 +300,8 @@ void InstrInfoEmitter::emitOperandNameMappings(
>      assert(MaxOperandNo <= INT16_MAX &&
>
> OS << "assert(NamesIdx < OPERAND_LAST);"; ?
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/llvm/llvm-project/pull/125424#pullrequestreview-2588627893>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/APRMUB6GVUYTMJT5UF6ITPD2NZ3SHAVCNFSM6AAAAABWKTEW26VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKOBYGYZDOOBZGM>
> .
> You are receiving this because your review was requested.Message ID:
> ***@***.***>
>


https://github.com/llvm/llvm-project/pull/125424