-
Notifications
You must be signed in to change notification settings - Fork 67
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
Preliminary Windows support #128
Conversation
It doesn't seem like it's possible to change the linker from within the build script, however, we can retrieve the linker in use and give the user a suggestion if the linker will not work.
wintendo user here approves |
The plan is to replace the stub generation by generating them with PHP code. This is cross-platform and means we don't need to worry about ABI. We also don't need to embed information into the library.
Some updates:
|
Hi, I just tried to build the hello_world example on my local Windows but i still get the error regarding the missing php-config. Do you have a hint for what I'm missing? |
Hey, It's working for me on windows. Otherwise please post the details of the errors. |
Still some work to do but will use this to track progress.
Some caveats:
__vectorcall
on some functions which is supported in Rust via theabi_vectorcall
nightly feature.__vectorcall
support has not been upstreamed yet and therefore requires using my fork.zend_fastcall!
macro. Function handlers are__vectorcall
on Windows and the macro will swap the ABI based on the target triple. Example: