[all-commits] [llvm/llvm-project] 14c578: [TableGen] Reduce size of MatchTableRecord (NFC) (...

[Nikita Popov via All-commits]

  Branch: refs/heads/release/20.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 14c5784ee4c03c5a7aa6f483486985d2c24a2407
      https://github.com/llvm/llvm-project/commit/14c5784ee4c03c5a7aa6f483486985d2c24a2407
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2025-02-07 (Fri, 07 Feb 2025)

  Changed paths:
    M llvm/utils/TableGen/Common/GlobalISel/GlobalISelMatchTable.cpp
    M llvm/utils/TableGen/Common/GlobalISel/GlobalISelMatchTable.h

  Log Message:
  -----------
  [TableGen] Reduce size of MatchTableRecord (NFC) (#125221)

MatchTableRecord stores a 64-bit RawValue. However, this field is only
needed by a small part of the code (jump table generation).

Create a separate RecordAndValue structure that is used in just the
necessary places.

Based on massif, this reduces memory usage on RISCVGenGlobalISel.inc by
about 100MB (to 2.15GB).

(cherry picked from commit e2301d674976b84ba505065a9702f3376e05bc43)


  Commit: 4d04a406a5a5e13af679018b116fc9975eb85579
      https://github.com/llvm/llvm-project/commit/4d04a406a5a5e13af679018b116fc9975eb85579
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2025-02-07 (Fri, 07 Feb 2025)

  Changed paths:
    M llvm/include/llvm/TableGen/Record.h

  Log Message:
  -----------
  [TableGen] Don't use inline storage for ReferenceLocs (NFC) (#125231)

The ReferenceLocs are not enabled by default (they are used by the
tablegen lsp server), and as such always empty, but still allocate
inline storage for the SmallVector. Disabling it saves about 200MB on
RISCVGenGlobalISel.inc.

(The equivalent field in Record already disables inline storage.)

(cherry picked from commit c640f97ccf723e64ff24af225cb995c905538406)


Compare: https://github.com/llvm/llvm-project/compare/3a3a3230d171...4d04a406a5a5

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