Skip to content
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

Add arch module #384

Closed
XAMPPRocky opened this issue Jan 18, 2021 · 2 comments
Closed

Add arch module #384

XAMPPRocky opened this issue Jan 18, 2021 · 2 comments
Labels
mcp: accepted A major change to the compiler that has been accepted.

Comments

@XAMPPRocky
Copy link
Member

Proposal

In the spirit of previous discussions of wanting to increase the capabilities of rust-gpu while also allowing more iteration to happen in crates outside of the main repo, I'd like to propose adding arch module to spirv-std, in which there is only single purpose unsafe functions that map to a single instruction in SPIR-V. This would allow more people to use features we currently don't support at any higher level sooner without having to write inline assembly themselves, while also giving us the time to design those abstractions without needing to rush them to provide any level of support in rust-gpu.

@XAMPPRocky XAMPPRocky added the mcp: proposed A major change to the compiler, that hasn't yet been approved. label Jan 18, 2021
@repi
Copy link
Contributor

repi commented Jan 21, 2021

Really like this proposal and idea and to enable more devs to directly access low-level "unsafe" intrinsics and functionality to experiment with what to build on top, and to help drive what higher-level ways we should expose functionality in our Rust GPU "std"

@XAMPPRocky XAMPPRocky added mcp: accepted A major change to the compiler that has been accepted. and removed mcp: proposed A major change to the compiler, that hasn't yet been approved. labels Jan 22, 2021
@XAMPPRocky
Copy link
Member Author

Yeah, based on the discussion in the previous meeting, we're generally in favour of this so I'm going to mark this as accepted, and close this issue. I'll make an initial PR adding some basic instructions and the rest can be tracked through #383

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mcp: accepted A major change to the compiler that has been accepted.
Projects
None yet
Development

No branches or pull requests

2 participants