-
Notifications
You must be signed in to change notification settings - Fork 187
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
Experimental AMD extensions #184
Conversation
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.
Looks pretty good to me. Do you also want to do the higher level loader? Also I just realized that if we split ash into ash-sys
then we can't easily extend StructureType
like this any more.
} | ||
} | ||
impl<'a> PhysicalDeviceWaveLimitPropertiesAmdBuilder<'a> { | ||
pub fn next<T>(mut self, next: &'a mut T) -> PhysicalDeviceWaveLimitPropertiesAmdBuilder<'a> |
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.
I believe this should be &'a T
right now, but if #183 merges then this is correct.
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.
Just leave it as it is right now.
Don't extensions need code in ash-sys anyway, for function signatures and struct types? Doesn't seem like this is any different to me. |
Yes it is not a real problem, we just have to move the "low level" part into |
Should be ready now. I skipped the high level API as I'm not sure if it would be of much worth for now |
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.
Looks good to me. Leaving out the high level wrapper is fine. Have you already played around with it?
A bit, mostly trying to make sense out of the exposed counters as 90% of these are not documented and GCN generation specific 😅 |
bors r+ |
184: Experimental AMD extensions r=MaikKlein a=msiglreith Addresses #178 (comment) Not fully supports all parts of the APIs, just trying to get some general feedback if this meets the expected structuring. Co-authored-by: msiglreith <[email protected]>
Addresses #178 (comment)
Not fully supports all parts of the APIs, just trying to get some general feedback if this meets the expected structuring.