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

[MSL] Metal Multi-View Rendering #2423

Open
bluesky013 opened this issue Dec 7, 2024 · 3 comments
Open

[MSL] Metal Multi-View Rendering #2423

bluesky013 opened this issue Dec 7, 2024 · 3 comments
Labels
help wanted Contribution from external parties is desired here.

Comments

@bluesky013
Copy link

When converting to MSL with multiview enabled, should amplification_id be used instead of layer rendering?

https://developer.apple.com/documentation/metal/render_passes/improving_rendering_performance_with_vertex_amplification?language=objc

@cdavis5e
Copy link
Contributor

cdavis5e commented Dec 7, 2024

I actually wanted to do this when I implemented multiview. The problem is that, at least at the time, vertex amplification only supported two views, and VK_KHR_multiview requires a minimum of six, so we'd still need to do some emulation, even with vertex amplification. One day, I'd like to go back to this and implement support for vertex amplification, but patches are always welcome.

@bluesky013
Copy link
Author

Thank you for your reply. According to the latest metal feature table, for chips above A14, the Maximum vertex count for vertex amplification can support up to 8.

1733620788640

If converted to layer rendering, it might pose challenges when handling different APIs such as Vulkan and Metal in the engine pipeline. This is because the biggest advantage of using MultiView is that it does not have a large impact on the organization of drawcalls in the pipeline, including cases where instancing/indirect usage exists.

@HansKristian-Work HansKristian-Work added the help wanted Contribution from external parties is desired here. label Dec 9, 2024
@HansKristian-Work
Copy link
Contributor

I guess whoever needs this should make a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Contribution from external parties is desired here.
Projects
None yet
Development

No branches or pull requests

3 participants