Remove sint/uint sampler check.
sint/uint textures can be used with textureGather
which takes a sampler.
This worked in release since this is a dcheck test
that gets compiled out but failed in other contexts.
Bug: 358407355
Fixed: 358407355
Change-Id: I0896224b1842e40ecb8f8109ee7e47cb3fd9dd65
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/203159
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Austin Eng <enga@chromium.org>
Auto-Submit: Gregg Tavares <gman@chromium.org>
diff --git a/src/dawn/native/ShaderModule.cpp b/src/dawn/native/ShaderModule.cpp
index 2c0fcba..69e3a64 100644
--- a/src/dawn/native/ShaderModule.cpp
+++ b/src/dawn/native/ShaderModule.cpp
@@ -1236,16 +1236,6 @@
const TextureBindingInfo& sampledTextureBindingInfo =
std::get<TextureBindingInfo>(textureInfo.bindingLayout);
- // Uint/Sint can't be statically used with a sampler, so they any
- // texture bindings reflected must be float or depth textures. If
- // the shader uses a float/depth texture but the bind group layout
- // specifies a uint/sint texture binding,
- // |ValidateCompatibilityWithBindGroupLayout| will fail since the
- // sampleType does not match.
- DAWN_ASSERT(sampledTextureBindingInfo.sampleType != wgpu::TextureSampleType::Undefined &&
- sampledTextureBindingInfo.sampleType != wgpu::TextureSampleType::Uint &&
- sampledTextureBindingInfo.sampleType != wgpu::TextureSampleType::Sint);
-
DAWN_INVALID_IF(
sampledTextureBindingInfo.sampleType == wgpu::TextureSampleType::UnfilterableFloat,
"Texture binding (group:%u, binding:%u) is %s but used statically with a sampler "