[llvm] [TableGen] Fix OperandMap table size (PR #125424)
Mariusz Sikora via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 3 00:04:34 PST 2025
mariusz-sikora-at-amd wrote:
> Yeah that will work. Though I am changing the code here in #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 <[#125424 (comment)](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 <[#125424 (review)](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: ***@***.***>
Please do. Thanks. I will close this PR.
https://github.com/llvm/llvm-project/pull/125424
More information about the llvm-commits
mailing list