Skip to content

Conversation

@kainino0x
Copy link
Collaborator

This has been standardized in the JS API, so we can add it now.

Small breaking change by adding items to WGPUAdapterInfo, but these items can always be populated, in the same way they are in JS, so they don't need to be in an extension struct.

Fixes #428

This has been standardized in the JS API, so we can add it now. Small breaking change by adding items to WGPUAdapterInfo, but these items can always be populated, in the same way they are in JS. Fixes 428
@kainino0x
Copy link
Collaborator Author

@dneto0 PTAL

@kainino0x
Copy link
Collaborator Author

@cwfitzgerald this is just matching the JS spec so I assume it's fine, but let me know if not

Copy link

@dneto0 dneto0 left a comment

Choose a reason for hiding this comment

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

LGTM!

@kainino0x kainino0x merged commit 4ec0698 into webgpu-native:main Jan 31, 2025
5 checks passed
@kainino0x kainino0x deleted the subgroups branch January 31, 2025 21:01
copybara-service bot pushed a commit to google/dawn that referenced this pull request Feb 4, 2025
Spec PR: webgpu-native/webgpu-headers#509 Change-Id: I5510a054304d802dfbcbfe1a03ef8f2e6db36032 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/223876 Reviewed-by: Alan Baker <alanbaker@google.com> Commit-Queue: Fr <beaufort.francois@gmail.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org>
copybara-service bot pushed a commit to google/dawn that referenced this pull request Feb 4, 2025
Spec PR: webgpu-native/webgpu-headers#509 Change-Id: I06e44901a363aca4903a3b5c577a6692b97f34f2 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/224214 Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Fr <beaufort.francois@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants