encounter
/
dawn-cmake
mirror of https://github.com/encounter/dawn-cmake.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
dawn-cmake/test/tint/shader_io/shared_struct_helper_functi...

96 lines
4.0 KiB
Plaintext
Raw Permalink Normal View History

test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
; SPIR-V
; Version: 1.3
; Generator: Google Tint Compiler; 0
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
; Bound: 46
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
; Schema: 0
OpCapability Shader
OpMemoryModel Logical GLSL450
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
OpEntryPoint Vertex %vert_main1 "vert_main1" %pos_1 %loc0_1 %vertex_point_size
OpEntryPoint Vertex %vert_main2 "vert_main2" %pos_2 %loc0_2 %vertex_point_size_1
OpName %pos_1 "pos_1"
OpName %loc0_1 "loc0_1"
OpName %vertex_point_size "vertex_point_size"
OpName %pos_2 "pos_2"
OpName %loc0_2 "loc0_2"
OpName %vertex_point_size_1 "vertex_point_size_1"
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpName %VertexOutput "VertexOutput"
OpMemberName %VertexOutput 0 "pos"
OpMemberName %VertexOutput 1 "loc0"
OpName %foo "foo"
OpName %x "x"
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
OpName %vert_main1_inner "vert_main1_inner"
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpName %vert_main1 "vert_main1"
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
OpName %vert_main2_inner "vert_main2_inner"
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpName %vert_main2 "vert_main2"
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
OpDecorate %pos_1 BuiltIn Position
OpDecorate %loc0_1 Location 0
OpDecorate %loc0_1 Flat
OpDecorate %vertex_point_size BuiltIn PointSize
OpDecorate %pos_2 BuiltIn Position
OpDecorate %loc0_2 Location 0
OpDecorate %loc0_2 Flat
OpDecorate %vertex_point_size_1 BuiltIn PointSize
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpMemberDecorate %VertexOutput 0 Offset 0
OpMemberDecorate %VertexOutput 1 Offset 16
%float = OpTypeFloat 32
%v4float = OpTypeVector %float 4
test: Add missing stage attributes to IO test Change-Id: I7f69de9c3b6fe226e0688f56a8d076dc6ec4a81f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55281 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 01:12:45 +00:00
%_ptr_Output_v4float = OpTypePointer Output %v4float
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%5 = OpConstantNull %v4float
%pos_1 = OpVariable %_ptr_Output_v4float Output %5
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
%int = OpTypeInt 32 1
test: Add missing stage attributes to IO test Change-Id: I7f69de9c3b6fe226e0688f56a8d076dc6ec4a81f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55281 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 01:12:45 +00:00
%_ptr_Output_int = OpTypePointer Output %int
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%9 = OpConstantNull %int
%loc0_1 = OpVariable %_ptr_Output_int Output %9
%_ptr_Output_float = OpTypePointer Output %float
%12 = OpConstantNull %float
%vertex_point_size = OpVariable %_ptr_Output_float Output %12
%pos_2 = OpVariable %_ptr_Output_v4float Output %5
%loc0_2 = OpVariable %_ptr_Output_int Output %9
%vertex_point_size_1 = OpVariable %_ptr_Output_float Output %12
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
%VertexOutput = OpTypeStruct %v4float %int
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%16 = OpTypeFunction %VertexOutput %float
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
%float_1 = OpConstant %float 1
%int_42 = OpConstant %int 42
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%25 = OpTypeFunction %VertexOutput
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
%float_0_5 = OpConstant %float 0.5
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%void = OpTypeVoid
%30 = OpTypeFunction %void
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
%float_0_25 = OpConstant %float 0.25
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%foo = OpFunction %VertexOutput None %16
test: Add missing stage attributes to IO test Change-Id: I7f69de9c3b6fe226e0688f56a8d076dc6ec4a81f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55281 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 01:12:45 +00:00
%x = OpFunctionParameter %float
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%20 = OpLabel
%22 = OpCompositeConstruct %v4float %x %x %x %float_1
%24 = OpCompositeConstruct %VertexOutput %22 %int_42
OpReturnValue %24
test: Add missing stage attributes to IO test Change-Id: I7f69de9c3b6fe226e0688f56a8d076dc6ec4a81f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55281 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 01:12:45 +00:00
OpFunctionEnd
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%vert_main1_inner = OpFunction %VertexOutput None %25
%27 = OpLabel
%28 = OpFunctionCall %VertexOutput %foo %float_0_5
OpReturnValue %28
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpFunctionEnd
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%vert_main1 = OpFunction %void None %30
test: Add missing stage attributes to IO test Change-Id: I7f69de9c3b6fe226e0688f56a8d076dc6ec4a81f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55281 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 01:12:45 +00:00
%33 = OpLabel
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%34 = OpFunctionCall %VertexOutput %vert_main1_inner
%35 = OpCompositeExtract %v4float %34 0
OpStore %pos_1 %35
%36 = OpCompositeExtract %int %34 1
OpStore %loc0_1 %36
OpStore %vertex_point_size %float_1
test: Add missing stage attributes to IO test Change-Id: I7f69de9c3b6fe226e0688f56a8d076dc6ec4a81f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55281 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 01:12:45 +00:00
OpReturn
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpFunctionEnd
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%vert_main2_inner = OpFunction %VertexOutput None %25
%38 = OpLabel
%39 = OpFunctionCall %VertexOutput %foo %float_0_25
OpReturnValue %39
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpFunctionEnd
spirv: Use generic transform to process shader IO The refactored CanonicalizeEntryPointIO transform makes it much easier to handle SPIR-V style IO as well, and doing this removes a lot of duplicated code. Remove all of the SPIR-V transform code for shader IO and vertex point size. Bug: tint:920 Change-Id: Id1b97517619b4d2fd09b45d5aee848259f3dfa77 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/60840 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Auto-Submit: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-08-05 17:34:19 +00:00
%vert_main2 = OpFunction %void None %30
%42 = OpLabel
%43 = OpFunctionCall %VertexOutput %vert_main2_inner
%44 = OpCompositeExtract %v4float %43 0
OpStore %pos_2 %44
%45 = OpCompositeExtract %int %43 1
OpStore %loc0_2 %45
OpStore %vertex_point_size_1 %float_1
test: Add missing stage attributes to IO test Change-Id: I7f69de9c3b6fe226e0688f56a8d076dc6ec4a81f Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/55281 Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: James Price <jrprice@google.com> Reviewed-by: Ben Clayton <bclayton@google.com>
2021-06-19 01:12:45 +00:00
OpReturn
test: Add E2E test coverage for shader IO This provides much more complete coverage than what we have in the unit tests. We now test: - All builtins, for all stages, both struct and non-struct - Multiple location attributes for vertex and fragment stages, both struct and non-struct - Mixing builtins and location attributes, whilst mixing struct and non-struct - A few "interesting" cases of IO structs being shared between different functions, stages, and with an SSBO variable There are 7 skipped tests for MSL due to two different MSL bugs which will be fixed in upcoming patches. Change-Id: I8b802591762c8ff018e01bf37838551e353162b1 Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/53120 Kokoro: Kokoro <noreply+kokoro@google.com> Reviewed-by: Ben Clayton <bclayton@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Auto-Submit: James Price <jrprice@google.com>
2021-06-03 09:38:34 +00:00
OpFunctionEnd