[all-commits] [llvm/llvm-project] 1442b0: [TableGen] Remove redundant buffer copies for ULEB...

[Jason Eckhardt via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1442b0e65370b603dcd4c7cfc300f19937c3bc79
      https://github.com/llvm/llvm-project/commit/1442b0e65370b603dcd4c7cfc300f19937c3bc79
  Author: Jason Eckhardt <jeckhardt at nvidia.com>
  Date:   2024-02-06 (Tue, 06 Feb 2024)

  Changed paths:
    M llvm/utils/TableGen/DecoderEmitter.cpp

  Log Message:
  -----------
  [TableGen] Remove redundant buffer copies for ULEB128 decode calls. (#80199)

This patch removes a couple of redundant buffer copies in emitTable for
setting up calls to decodeULEB128. Instead, provide the Table.data
buffer directly to the calls-- where decodeULEB128 does its own buffer
overflow checking.

Factor out 7 explicit loops to emit ULEB128 bytes into emitULEB128. Also
factor out 4 copies of 24-bit numtoskip emission into emitNumToSkip.

The functionality is already covered by existing unit tests and by
virtue of most of the in-tree back-ends exercising the decoder emitter.