Skip to content

Add textureBindingViewDimension and compat limits#580

Merged
kainino0x merged 6 commits intowebgpu-native:mainfrom
beaufortfrancois:compat
Mar 9, 2026
Merged

Add textureBindingViewDimension and compat limits#580
kainino0x merged 6 commits intowebgpu-native:mainfrom
beaufortfrancois:compat

Conversation

@beaufortfrancois
Copy link
Contributor

@beaufortfrancois beaufortfrancois commented Feb 13, 2026

Fixes #470

@beaufortfrancois
Copy link
Contributor Author

@kainino0x Please review

@beaufortfrancois beaufortfrancois changed the title Add GPUTextureDescriptor textureBindingViewDimension Add textureBindingViewDimension and compat limits Feb 13, 2026
@beaufortfrancois
Copy link
Contributor Author

I'll wait for #582 and #583 to be merged before resuming my work on this PR.

kainino0x added a commit that referenced this pull request Feb 20, 2026
Replace the manually-maintained INIT test lines with autogenerated
tests. Used a quick hack in perl (only tool I could find that could do
this easily), it just finds all the INIT macros defined in the header
and outputs a test for each one.

It's checked into the repo (instead of generated during build) so that I
don't have to deal with making it work on the Windows compile test.

This should have caught some missing items in
#580
kainino0x added a commit that referenced this pull request Feb 20, 2026
This is currently unused, but will get used in #580. Not easy to add
specific tests for this so I'm leaving it untested until that lands.
I've tested it locally with the changes I pushed there:
afba5c5
@beaufortfrancois
Copy link
Contributor Author

Are we good merging now?

@kainino0x
Copy link
Collaborator

Need review on #584 first. cwfitzgerald will look at it tomorrow

@Kangz
Copy link
Collaborator

Kangz commented Mar 6, 2026

Up @cwfitzgerald please take a look.

@cwfitzgerald
Copy link
Collaborator

Sorry for the delay, I've approved #584.

Copy link
Collaborator

@kainino0x kainino0x left a comment

Choose a reason for hiding this comment

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

LGTM, this should be safe to merge, it just copies the standardized JS API except for the naming of the structs, which I think follows the conventions we've established.

@kainino0x kainino0x enabled auto-merge (squash) March 9, 2026 20:30
@kainino0x kainino0x merged commit 3eb62ed into webgpu-native:main Mar 9, 2026
5 checks passed
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.

Add new limits added for Compatibility Mode

4 participants