elliottt opened PR #4496 from trevor/x64-inst-size
to main
:
- Box SyntheticAmode
- Box label vectors
- Switch VecMachLabel to a smallvec of 16 elements
- x64 instructions are now 40 bytes
<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
elliottt edited PR #4496 from trevor/x64-inst-size
to main
:
Experimenting with shrinking down the size of
Instr
for the x64 backend. The current state of this PR is that it shrinks the size ofInstr
to 40 bytes, but costs some compile performance as it puts every SyntheticAmode behind a box. This isn't really acceptable, so I'm going to investigate a new representation for SyntheticAmode that doesn't require 20 bytes.
<!--Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
Last updated: Nov 22 2024 at 16:03 UTC