Skip to content

Windows (MSVC) bootstrapping script #140

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

Merged
merged 13 commits into from
Jan 6, 2023
Merged

Conversation

kcbanner
Copy link
Contributor

@kcbanner kcbanner commented Nov 11, 2022

Using build.bat, you can start with MSVC (and the msvc bundled cmake/ninja), and end up with an x64_86-windows-gnu zig.

I've pulled in this change: https://reviews.llvm.org/D134165 in order to fix up how the .def files are passed. The branch we need here is the elseif(MINGW) one.

This build script depends on changes in these pull requests: ziglang/zig#13514 ziglang/zig#14002

add failure checks
fixup paths
…use the alternate compiler with the visual studio generator

fix variable quoting
- Disabled building targets with msvc  ABI
- Added build options for supporting the msvc  ABI, once support lands in the linker
- Build zlib with the same cflags as zig, so the runtimes are compatible when built with msvc
- Remove -Denable-stage1
@kcbanner kcbanner marked this pull request as ready for review December 31, 2022 19:07
@kcbanner
Copy link
Contributor Author

Moving into review now that ziglang/zig#14002 and ziglang/zig#13514 are in review.

@andrewrk
Copy link
Member

andrewrk commented Jan 2, 2023

I see that the patch has landed upstream in the main branch in llvm/llvm-project@dfbaf90, so it's OK for us to carry that patch until we update to LLVM 16 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants