Skip to content

[breaking] Add new texture formats from texture-formats-tier1#553

Merged
kainino0x merged 1 commit into
webgpu-native:mainfrom
AlbinBernhardssonARM:texture-formats-tier1
Jul 7, 2025
Merged

[breaking] Add new texture formats from texture-formats-tier1#553
kainino0x merged 1 commit into
webgpu-native:mainfrom
AlbinBernhardssonARM:texture-formats-tier1

Conversation

@AlbinBernhardssonARM
Copy link
Copy Markdown
Contributor

Add the new texture formats from texture-formats-tier1 (16 bits per channel unorm/snorm). Maintain the same ordering as in the JS API. This changes the ABI by changing enum values.

Add the new texture formats from texture-formats-tier1 (16 bits per
channel unorm/snorm). Maintain the same ordering as in the JS API. This
changes the ABI by changing enum values.
Copy link
Copy Markdown
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, thank you!

BTW, I think we should stop changing enum values about now - for the FeatureName I mostly did it so compat would be first, but it's also nice for these to be in a less confusing order. Since it probably shouldn't affect anyone in practice I say let's go ahead with this.

@kainino0x kainino0x enabled auto-merge (squash) July 7, 2025 23:33
@kainino0x kainino0x merged commit 5a6c263 into webgpu-native:main Jul 7, 2025
5 checks passed
copybara-service Bot pushed a commit to google/dawn that referenced this pull request Jul 12, 2025
Also updates TextureUtils.cpp, including accounting for
TextureFormatsTier1 on both Dawn and Emscripten, and fixing a bug that
mixed up Snorm16 formats with wgpu::FeatureName::Unorm16TextureFormats.

Upstream WGPUTextureFormat change:
webgpu-native/webgpu-headers#553

Roll also eliminates the diff on wgpuInstanceGetWGSLLanguageFeatures.

Bug: 429178774, 427657664, 421941589
Change-Id: I4a3e88ab8a902d9087d69b3f36a32f8a685a60c2
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/252260
Reviewed-by: Loko Kung <lokokung@google.com>
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
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.

2 participants