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 support for VK_VALVE_mutable_descriptor_type #375

Merged
merged 3 commits into from Dec 7, 2020

Conversation

HansKristian-Work
Copy link
Owner

@HansKristian-Work HansKristian-Work commented Nov 9, 2020

@@ -754,6 +767,7 @@ static void vkd3d_physical_device_info_init(struct vkd3d_physical_device_info *i
VkPhysicalDeviceCustomBorderColorFeaturesEXT *custom_border_color_features;
VkPhysicalDeviceTimelineSemaphoreFeaturesKHR *timeline_semaphore_features;
VkPhysicalDevicePushDescriptorPropertiesKHR *push_descriptor_properties;
VkPhysicalDeviceMutableDescriptorTypeFeaturesVALVE *mutable_features;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

funny christmastree ordering

@HansKristian-Work HansKristian-Work force-pushed the mutable-descriptor branch 5 times, most recently from 69ad463 to 3cbda66 Compare November 25, 2020 19:24
@HansKristian-Work HansKristian-Work force-pushed the mutable-descriptor branch 2 times, most recently from e5ff894 to daab2cd Compare November 27, 2020 15:55
@HansKristian-Work HansKristian-Work force-pushed the mutable-descriptor branch 3 times, most recently from 3e6191d to 5090298 Compare December 7, 2020 12:35
@HansKristian-Work HansKristian-Work marked this pull request as ready for review December 7, 2020 12:35
@HansKristian-Work HansKristian-Work changed the title WIP: Add mutable descriptor support. Add support for VK_VALVE_mutable_descriptor_type Dec 7, 2020
@baryluk
Copy link

baryluk commented Dec 7, 2020

Hi Hans, I can test this Vulkan, Mesa and vkd3d changes on my AMD Fury X (FIJI, GFX8). Are there some specific game titles to test this with that you would recommend? For testing correctness and performance compared to without it.

Copy link
Collaborator

@doitsujin doitsujin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

few nits and questions.

libs/vkd3d/resource.c Outdated Show resolved Hide resolved
libs/vkd3d/resource.c Outdated Show resolved Hide resolved
libs/vkd3d/state.c Outdated Show resolved Hide resolved
libs/vkd3d/state.c Outdated Show resolved Hide resolved
Signed-off-by: Hans-Kristian Arntzen <post@arntzen-software.no>
Signed-off-by: Hans-Kristian Arntzen <post@arntzen-software.no>
Signed-off-by: Hans-Kristian Arntzen <post@arntzen-software.no>
@HansKristian-Work HansKristian-Work merged commit 7711b9b into master Dec 7, 2020
@HansKristian-Work HansKristian-Work deleted the mutable-descriptor branch December 7, 2020 14:17
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.

None yet

4 participants