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

rewrite: Initial codegen #867

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

rewrite: Initial codegen #867

wants to merge 3 commits into from

Conversation

Friz64
Copy link
Contributor

@Friz64 Friz64 commented Feb 29, 2024

This is based on #745 and eddyb's C parsing PR (Friz64#1). This PR will be rebased once these are merged.

I only implemented the most basic of struct outputs to have something to work with, extending that is the next step.

The directory structure used here is inspired by erupt. All Vulkan items still get re-exported into ash::vk though.


This PR is the product of this livestream: https://www.youtube.com/watch?v=wgSMfX_QcSw (very unentertaining content, yet it definitely helped to overcome motivation issues)

@Friz64 Friz64 marked this pull request as ready for review February 29, 2024 01:26
@Friz64 Friz64 force-pushed the structures branch 4 times, most recently from 6b0735d to 054ed82 Compare March 13, 2024 22:34
@stefnotch
Copy link

This looks very promising.
On that note, I would be happy to assist if there's anything you need, from small testing tasks to implementing some tricky features.

@Friz64
Copy link
Contributor Author

Friz64 commented Mar 16, 2024

Sure, the next step in my mind would be to actually implement struct/union generation. For that we need lifetime propagation.

I think it would be convenient to be able to discuss this in some kind of group chat for ash development. I'd usually use Discord for this, but I'm open to suggestions as I'm not adamant on it.

@MarijnS95
Copy link
Collaborator

@Friz64 there is a gitter instance for this repository. Link should be in the readme, it's using the old repo url because gitter couldn't migrate.

You can use various clients like Matrix to connect.

@Friz64 Friz64 marked this pull request as draft July 10, 2024 21:37
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.

4 participants