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 optional support for VkPhysicalDeviceFeatures::wideLines. #2137

Merged
merged 1 commit into from
Jan 26, 2024

Conversation

billhollings
Copy link
Contributor

@billhollings billhollings commented Jan 25, 2024

  • Add MVK_USE_METAL_PRIVATE_API build setting to allow MoltenVK to be built with access to Metal private API calls.
  • Add support for VkPhysicalDeviceFeatures::wideLines feature when MVK_USE_METAL_PRIVATE_API is enabled in a MoltenVK build.
  • Set lineWidthRange and lineWidthGranularity to reasonable arbitrary values.

Fixes #1657.
Fixes #1670.
Replaces #2091.

Future PR's can leverage MVK_USE_METAL_PRIVATE_API to add additional private functionality, as it is uncovered.

MoltenVK/MoltenVK/Commands/MVKCommandEncoderState.h Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
- Add MVK_USE_METAL_PRIVATE_API build setting to allow MoltenVK
  to be built with access to Metal private API calls.
- Add support for VkPhysicalDeviceFeatures::wideLines feature
  when MVK_USE_METAL_PRIVATE_API is enabled in a MoltenVK build.
- Set lineWidthRange and lineWidthGranularity to reasonable arbitrary values.
Copy link
Collaborator

@cdavis5e cdavis5e left a comment

Choose a reason for hiding this comment

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

Dang, you beat me to the punch. :)

I actually have similar changes in my own tree, complete with settings to enable/disable them at build time and at run time. After this one goes in, I'm going to try to get my other changes upstreamed.

Docs/Whats_New.md Show resolved Hide resolved
MoltenVK/MoltenVK/Commands/MVKCmdRendering.h Show resolved Hide resolved
MoltenVK/MoltenVK/Utility/MVKEnvironment.h Show resolved Hide resolved
@billhollings
Copy link
Contributor Author

Dang, you beat me to the punch. :)

I hate when that happens! 😖

To help with this risk, I try to either assign myself to the related issue (although I didn't do that here because it was a quick fix), or add it to the upcoming task list (which I did do on this one).

But I'm open to other mechanisms for us to communicate what's being worked on. It would be good to have some mechanism that we can all collaborate on.

I actually have similar changes in my own tree, complete with settings to enable/disable them at build time and at run time. After this one goes in, I'm going to try to get my other changes upstreamed.

Okay. Looking forward to seeing it.

@billhollings billhollings merged commit b5ffb47 into KhronosGroup:main Jan 26, 2024
6 checks passed
@billhollings billhollings deleted the private-widelines branch January 26, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants