-
Notifications
You must be signed in to change notification settings - Fork 152
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
Initial SPIRV no_std implementation #85
Conversation
I'm not sure it's of use to your SPIRV efforts but I fixed It requires |
@bitshifter thanks will check out! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean I can't printf debug on the GPU :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Wooho! 🎉🚀 |
This is a first early test implementation for our new
spriv-unknown-unknown
target that we have in our Rust GPU compiler backend and to bring upglam
to support it, this overall effort is tracked and discussed in EmbarkStudios/rust-gpu#134We currently require
no_std
so this enablesno_std
(related discussion: #5) when building glam for that target but then defines a small extension trait to bring in thef32
functionality fromstd
. This is right now defined directly inglam
here in a separatespirv.rs
and is missing some intrinsic liketan
that we'll need we'll need to add to Rust core.Pretty happy that did get most things to compile this way, but some glam functionality has been stubbed out. Not too complex.
Expect this to live as a draft PR here for quite a while until we have settled on some of these intrinsics implementations, where they should live, and fixed some compiler backend issues with the this for now also. But having it as an open draft PR makes it more visible and easier to discuss approaches.