-
-
Notifications
You must be signed in to change notification settings - Fork 219
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
Support llvm distribution for windows #4
Comments
The main problem with this issue is that I do not have access to a Windows machine to test. And it looks like it will be a fair bit of work to test everything in a non-Unix setting. I am afraid I will have to rely on the community for this feature implementation. |
I started taking a look at this (filed #13 in the process), but there's a problematic issue: LLVM for Windows is released only as a installable package, not a simple archive - or at least I didn't find any on the LLVM website. |
Yes, I am not sure how to deal with this either. Can we install the package to a given directory path? Even if is a system default path, I don't see particular harm in installing a specific version of LLVM if the install does not overwrite installs of other versions of LLVM. |
I also came across the requirement for clang under windows. |
We don't have to rely on download_and_extract. We can always execute the self-extracting archive in the repository rule. |
I have developed a different prototype patch series that works for Windows (I didn't notice the one developed by @zaucy until just now). It gets around needing to run the self-extracting executable, by relying on a re-packaged compressed archive instead of shelling out to a separate 7zip install. For my demonstration purposes, I just attached it as a release artifact to a github release. https://github.com/mjbots/bazel-toolchain/commits/windows The patch sequence isn't suited for submission yet, but if the general approach seemed reasonable, I would be happy to repackage it. I also have a semi-independent ability to build libc++ from source and use that with binaries instead of the msvc standard library. That let me run coroutines on Windows with clang 10.0. It is less optimal, since I haven't figured out bazel toolchain transitions well enough to make it seamless yet. If you use that feature, every |
If people are still interested in this, I think we can get in a hacky thing in and then make it more proper over time. I don't use a Windows machine myself, so if anyone wants to put in effort, I can give them write access and let them manage the Windows feature. |
I'm also interested in this feature. And llvm v18.1.1 is released https://github.com/llvm/llvm-project/releases/tag/llvmorg-18.1.1 and clang+llvm-18.1.1-x86_64-pc-windows-msvc.tar.xz is officially supported. Is it possible to take a look at Windows support for now? |
* add void * try this version * whoops --------- Co-authored-by: Getty Ritter <[email protected]>
subj
The text was updated successfully, but these errors were encountered: