test/tint/builtins/gen: Use returned value
It appears that FXC and DXC do some validation post dead-code-elimination.
These tests have been updated so that the return value is assigned to a storage buffer, ensuring that all validation is performed.
Many DXC tests are affected by https://github.com/microsoft/DirectXShaderCompiler/issues/5082, which have been SKIP'ed.
Fixed: tint:1859
Change-Id: I0001a9a9821846cd0855c3d8ce2bec79ab8e64ef
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/122662
Kokoro: Kokoro <noreply+kokoro@google.com>
Reviewed-by: James Price <jrprice@google.com>
Commit-Queue: James Price <jrprice@google.com>
Auto-Submit: Ben Clayton <bclayton@google.com>
diff --git a/test/tint/builtins/gen/var/asin/11dfda.wgsl.expected.glsl b/test/tint/builtins/gen/var/asin/11dfda.wgsl.expected.glsl
index 466612a..a72fec2 100644
--- a/test/tint/builtins/gen/var/asin/11dfda.wgsl.expected.glsl
+++ b/test/tint/builtins/gen/var/asin/11dfda.wgsl.expected.glsl
@@ -1,9 +1,14 @@
#version 310 es
#extension GL_AMD_gpu_shader_half_float : require
+layout(binding = 0, std430) buffer prevent_dce_block_ssbo {
+ float16_t inner;
+} prevent_dce;
+
void asin_11dfda() {
float16_t arg_0 = 0.479248046875hf;
float16_t res = asin(arg_0);
+ prevent_dce.inner = res;
}
vec4 vertex_main() {
@@ -23,9 +28,14 @@
#extension GL_AMD_gpu_shader_half_float : require
precision mediump float;
+layout(binding = 0, std430) buffer prevent_dce_block_ssbo {
+ float16_t inner;
+} prevent_dce;
+
void asin_11dfda() {
float16_t arg_0 = 0.479248046875hf;
float16_t res = asin(arg_0);
+ prevent_dce.inner = res;
}
void fragment_main() {
@@ -39,9 +49,14 @@
#version 310 es
#extension GL_AMD_gpu_shader_half_float : require
+layout(binding = 0, std430) buffer prevent_dce_block_ssbo {
+ float16_t inner;
+} prevent_dce;
+
void asin_11dfda() {
float16_t arg_0 = 0.479248046875hf;
float16_t res = asin(arg_0);
+ prevent_dce.inner = res;
}
void compute_main() {
