Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ircode: cleanup code crud #56724

Merged
merged 3 commits into from
Dec 4, 2024
Merged

ircode: cleanup code crud #56724

merged 3 commits into from
Dec 4, 2024

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented Nov 30, 2024

  • support gc running
  • don't duplicate field 4
  • remove some unused code only previously needed for handling cycles (which are not valid in IR)
  • avoid serializing ssaflags, which are very large (relative to the rest of the IR) and usually all zeros

sysimg : 159 MB => 157 MB => 154 MB
du -sh usr/share/julia/compiled : 260M => 256 MB => 254 MB

 - support gc running
 - don't duplicate field 4
 - remove some unused code only previously needed for handling cycles
   (which are not valid in IR)
Since most ssavalue are used just after their def, this gives a small
memory savings on compressed IR (a fraction of a percent).
…ll zero

When not all-zero, run-length encoding would also probably be great
here for lowered code (before inference).
@vtjnash vtjnash added the don't squash Don't squash merge label Dec 3, 2024
@vtjnash vtjnash merged commit 35976df into master Dec 4, 2024
8 of 9 checks passed
@vtjnash vtjnash deleted the jn/ircode-cleanup branch December 4, 2024 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
don't squash Don't squash merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant