[PATCH] D149445: [MachineInst] Switch NumOperands to 16bits

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 09:03:12 PDT 2023


jroelofs added a comment.

This change broke the build for an internal project of ours (breadcrumb: rdar://109362033). The relevant code being compiled is a fairly large nested switch that results in a PHI node with 65k+ operands, which can't easily be turned into a table for perf reasons.

WDYT about making it the lower 24 bits of a `uint32_t` bitfield shared with `AsmPrinterFlags` in the upper 8?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149445/new/

https://reviews.llvm.org/D149445



More information about the llvm-commits mailing list