Skip to content

Update Zig and use the x86_64 backend and incremental compilation #1583

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Update Zig and use the x86_64 backend and incremental compilation #1583

wants to merge 2 commits into from

Conversation

IntegratedQuantum
Copy link
Member

@IntegratedQuantum IntegratedQuantum commented Jun 2, 2025 •
edited
Loading

  • Enable the x86_64 backend in debug mode (done in Zig master)
  • Fix graphics issues with x86_64 enabled (→caused by Vector sizes are different in x86_64 backend ziglang/zig#24179)
  • Fix crash when opening the debug window with x86_64 enabled
  • Figure out a way to work around the awful performance problems of the x86_64 backend output, it takes literally a minute to load the world (→ these are all caused by memcpy from array access, this is quite frequent and I don't want to patch it in all locations, so we'll need to wait for strictly better -ODebug machine code quality than LLVM ziglang/zig#24144)
  • Upload a new zig version to our mirror
  • Check if incremental compilation works and enable it
  • Use the new vector and support in models.zig

@IntegratedQuantum
Copy link
Member Author

IntegratedQuantum commented Jun 14, 2025 •
edited
Loading

Currently blocked by ziglang/zig#24179

@IntegratedQuantum IntegratedQuantum mentioned this pull request Jun 19, 2025
Open
3 tasks
@IntegratedQuantum
Copy link
Member Author

IntegratedQuantum commented Jun 19, 2025 •
edited
Loading

Currently blocked by ziglang/zig#24144, particularly waiting on better array access (which I believe is tracked by ziglang/zig#24183).

@IntegratedQuantum IntegratedQuantum marked this pull request as draft July 10, 2025 18:53
@IntegratedQuantum IntegratedQuantum mentioned this pull request Jul 11, 2025
Merged
4 tasks
IntegratedQuantum added a commit that referenced this pull request Jul 15, 2025
@IntegratedQuantum
Update Zig to use the new Writergate version (#1664)
aae66ea 
For reference: ziglang/zig#24329

some commits have been extracted from #1583, but the x86_64 backend has
been disabled due to its horrible performance.

Remaining work:
- [x] Wait for official builds on ziglang.org and upload them to our
repository
- [x] Add workaround for ziglang/zig#24466
- [x] Fix TODO comment about ANSI support in stdout
- [x] Check for compile-time performance changes → it went from 13.1 to
11.9 seconds 🎉
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@IntegratedQuantum