[PATCH] D96317: [TableGen][SelectionDAG] Improve efficiency of encoding negative immediates for isel's CheckInteger opcode.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 10:54:51 PST 2021


craig.topper added a comment.

In D96317#2550753 <https://reviews.llvm.org/D96317#2550753>, @RKSimon wrote:

>> This gives a small reduction in the table size on all in tree targets except VE where size increased by about 300 bytes due to intrinsic ids now requiring 3 bytes instead of 2. Since the intrinsic enum space is shared by all targets this an unfortunate consquence of where VE is currently located in the range.
>
> I'm not very familiar with this - what options do we have for altering the range to help avoid the VE bloat?

Pretty sure the intrinsic IDs are just alphabetized and they are all prefixed by the target name. So I think it's just a function of how many target names come before "VE" and how many intrinsics those targets have.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96317



More information about the llvm-commits mailing list