Remove storage textures from vertex tests
The template change in test/tint/builtins/gen/gen.wgsl.tmpl removes the
remaining usages of storage textures in vertex shaders in Tint end2end
tests, replacing them instead with a VertexOutput object to prevent DCE.
The template is the only manually modified file in this change, the rest
were created with:
./tools/run gen
./tools/run tests --generate-expected
Binding numbers and entry point ordering had to be slightly shuffled to
work around: crbug.com/42250109
Bug: 344846829
Change-Id: I6c2c80b78168a13c6c545e7a0dc924d64997ff0e
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/193260
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Commit-Queue: Natalie Chouinard <chouinard@google.com>
diff --git a/test/tint/builtins/gen/literal/textureLoad/d80ff3.wgsl.expected.wgsl b/test/tint/builtins/gen/literal/textureLoad/d80ff3.wgsl.expected.wgsl
index e897c87..e568913 100644
--- a/test/tint/builtins/gen/literal/textureLoad/d80ff3.wgsl.expected.wgsl
+++ b/test/tint/builtins/gen/literal/textureLoad/d80ff3.wgsl.expected.wgsl
@@ -1,24 +1,33 @@
@group(1) @binding(0) var arg_0 : texture_storage_1d<bgra8unorm, read_write>;
-fn textureLoad_d80ff3() {
+fn textureLoad_d80ff3() -> vec4<f32> {
var res : vec4<f32> = textureLoad(arg_0, 1i);
- prevent_dce = res;
+ return res;
}
-@group(2) @binding(0) var<storage, read_write> prevent_dce : vec4<f32>;
-
-@vertex
-fn vertex_main() -> @builtin(position) vec4<f32> {
- textureLoad_d80ff3();
- return vec4<f32>();
-}
+@group(0) @binding(0) var<storage, read_write> prevent_dce : vec4<f32>;
@fragment
fn fragment_main() {
- textureLoad_d80ff3();
+ prevent_dce = textureLoad_d80ff3();
}
@compute @workgroup_size(1)
fn compute_main() {
- textureLoad_d80ff3();
+ prevent_dce = textureLoad_d80ff3();
+}
+
+struct VertexOutput {
+ @builtin(position)
+ pos : vec4<f32>,
+ @location(0) @interpolate(flat)
+ prevent_dce : vec4<f32>,
+}
+
+@vertex
+fn vertex_main() -> VertexOutput {
+ var out : VertexOutput;
+ out.pos = vec4<f32>();
+ out.prevent_dce = textureLoad_d80ff3();
+ return out;
}
