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
Determine GPU type on DX11 when possible #3490
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.
Nicely done!
Could you add a commit that bumps the backend version and adds a note to CHANGELOG, please?
Added 👍 |
4ca7131
to
5077757
Compare
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.
Looking good! A couple of small logic issues but we're almost there :)
5077757
to
a99099f
Compare
Updated |
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.
One more nit then we're good to go!
a99099f
to
c2c960b
Compare
Thanks! bors r+ |
@Diggsey please make the same PR against |
3485: Enable vulkan debug markers in release builds r=kvark a=fintelia Fixes #3482 (at least for the Vulkan backend) PR checklist: - [x] `make` succeeds (on *nix) - [ ] `make reftests` succeeds - [ ] tested examples with the following backends: 3492: Determine GPU type on DX11 when possible r=kvark a=Diggsey Cherry pick #3490 to master Co-authored-by: Jonathan Behrens <fintelia@gmail.com> Co-authored-by: Diggory Blake <diggsey@googlemail.com>
3485: Enable vulkan debug markers in release builds r=kvark a=fintelia Fixes #3482 (at least for the Vulkan backend) PR checklist: - [x] `make` succeeds (on *nix) - [ ] `make reftests` succeeds - [ ] tested examples with the following backends: 3492: Determine GPU type on DX11 when possible r=kvark a=Diggsey Cherry pick #3490 to master 3493: [master/dx11] Increase max bound descriptor set count r=kvark a=cwfitzgerald Rolls up #3487 now that wgpu is targeting master! Should be the last major rollup I need to do! Woooo! I looked at #3449 again but I don't think master needs that change, it looks correct. Co-authored-by: Jonathan Behrens <fintelia@gmail.com> Co-authored-by: Diggory Blake <diggsey@googlemail.com> Co-authored-by: Connor Fitzgerald <connorwadefitzgerald@gmail.com>
Partially addresses #3489
From what I can tell there's no way to discover this via the DX11 APIs, so guess based on the vendor and device IDs. This can be expanded on later, but initially I've just set it to guess "integrated" for Intel GPUs and "discrete" for everything else, which probably covers >90% of the cases where this applies...