| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408 |
- // Copyright (c) 2022 Google LLC
- //
- // Licensed under the Apache License, Version 2.0 (the "License");
- // you may not use this file except in compliance with the License.
- // You may obtain a copy of the License at
- //
- // http://www.apache.org/licenses/LICENSE-2.0
- //
- // Unless required by applicable law or agreed to in writing, software
- // distributed under the License is distributed on an "AS IS" BASIS,
- // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- // See the License for the specific language governing permissions and
- // limitations under the License.
- #include <iostream>
- #include "test/opt/pass_fixture.h"
- #include "test/opt/pass_utils.h"
- namespace spvtools {
- namespace opt {
- namespace {
- using InterfaceVariableScalarReplacementTest = PassTest<::testing::Test>;
- TEST_F(InterfaceVariableScalarReplacementTest,
- ReplaceInterfaceVarsWithScalars) {
- const std::string spirv = R"(
- OpCapability Shader
- OpCapability Tessellation
- OpMemoryModel Logical GLSL450
- OpEntryPoint TessellationControl %func "shader" %x %y %z %w %u %v
- ; CHECK: OpName [[x:%\w+]] "x"
- ; CHECK-NOT: OpName {{%\w+}} "x"
- ; CHECK: OpName [[y:%\w+]] "y"
- ; CHECK-NOT: OpName {{%\w+}} "y"
- ; CHECK: OpName [[z0:%\w+]] "z"
- ; CHECK: OpName [[z1:%\w+]] "z"
- ; CHECK: OpName [[w0:%\w+]] "w"
- ; CHECK: OpName [[w1:%\w+]] "w"
- ; CHECK: OpName [[u0:%\w+]] "u"
- ; CHECK: OpName [[u1:%\w+]] "u"
- ; CHECK: OpName [[v0:%\w+]] "v"
- ; CHECK: OpName [[v1:%\w+]] "v"
- ; CHECK: OpName [[v2:%\w+]] "v"
- ; CHECK: OpName [[v3:%\w+]] "v"
- ; CHECK: OpName [[v4:%\w+]] "v"
- ; CHECK: OpName [[v5:%\w+]] "v"
- OpName %x "x"
- OpName %y "y"
- OpName %z "z"
- OpName %w "w"
- OpName %u "u"
- OpName %v "v"
- ; CHECK-DAG: OpDecorate [[x]] Location 2
- ; CHECK-DAG: OpDecorate [[y]] Location 0
- ; CHECK-DAG: OpDecorate [[z0]] Location 0
- ; CHECK-DAG: OpDecorate [[z0]] Component 0
- ; CHECK-DAG: OpDecorate [[z1]] Location 1
- ; CHECK-DAG: OpDecorate [[z1]] Component 0
- ; CHECK-DAG: OpDecorate [[z0]] Patch
- ; CHECK-DAG: OpDecorate [[z1]] Patch
- ; CHECK-DAG: OpDecorate [[w0]] Location 2
- ; CHECK-DAG: OpDecorate [[w0]] Component 0
- ; CHECK-DAG: OpDecorate [[w1]] Location 3
- ; CHECK-DAG: OpDecorate [[w1]] Component 0
- ; CHECK-DAG: OpDecorate [[w0]] Patch
- ; CHECK-DAG: OpDecorate [[w1]] Patch
- ; CHECK-DAG: OpDecorate [[u0]] Location 3
- ; CHECK-DAG: OpDecorate [[u0]] Component 2
- ; CHECK-DAG: OpDecorate [[u1]] Location 4
- ; CHECK-DAG: OpDecorate [[u1]] Component 2
- ; CHECK-DAG: OpDecorate [[v0]] Location 3
- ; CHECK-DAG: OpDecorate [[v0]] Component 3
- ; CHECK-DAG: OpDecorate [[v1]] Location 4
- ; CHECK-DAG: OpDecorate [[v1]] Component 3
- ; CHECK-DAG: OpDecorate [[v2]] Location 5
- ; CHECK-DAG: OpDecorate [[v2]] Component 3
- ; CHECK-DAG: OpDecorate [[v3]] Location 6
- ; CHECK-DAG: OpDecorate [[v3]] Component 3
- ; CHECK-DAG: OpDecorate [[v4]] Location 7
- ; CHECK-DAG: OpDecorate [[v4]] Component 3
- ; CHECK-DAG: OpDecorate [[v5]] Location 8
- ; CHECK-DAG: OpDecorate [[v5]] Component 3
- OpDecorate %z Patch
- OpDecorate %w Patch
- OpDecorate %z Location 0
- OpDecorate %x Location 2
- OpDecorate %v Location 3
- OpDecorate %v Component 3
- OpDecorate %y Location 0
- OpDecorate %w Location 2
- OpDecorate %u Location 3
- OpDecorate %u Component 2
- %uint = OpTypeInt 32 0
- %uint_1 = OpConstant %uint 1
- %uint_2 = OpConstant %uint 2
- %uint_3 = OpConstant %uint 3
- %uint_4 = OpConstant %uint 4
- %_arr_uint_uint_2 = OpTypeArray %uint %uint_2
- %_ptr_Output__arr_uint_uint_2 = OpTypePointer Output %_arr_uint_uint_2
- %_ptr_Input__arr_uint_uint_2 = OpTypePointer Input %_arr_uint_uint_2
- %_ptr_Input_uint = OpTypePointer Input %uint
- %_ptr_Output_uint = OpTypePointer Output %uint
- %_arr_arr_uint_uint_2_3 = OpTypeArray %_arr_uint_uint_2 %uint_3
- %_ptr_Input__arr_arr_uint_uint_2_3 = OpTypePointer Input %_arr_arr_uint_uint_2_3
- %_arr_arr_arr_uint_uint_2_3_4 = OpTypeArray %_arr_arr_uint_uint_2_3 %uint_4
- %_ptr_Output__arr_arr_arr_uint_uint_2_3_4 = OpTypePointer Output %_arr_arr_arr_uint_uint_2_3_4
- %_ptr_Output__arr_arr_uint_uint_2_3 = OpTypePointer Output %_arr_arr_uint_uint_2_3
- %z = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %x = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %y = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- %w = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- %u = OpVariable %_ptr_Input__arr_arr_uint_uint_2_3 Input
- %v = OpVariable %_ptr_Output__arr_arr_arr_uint_uint_2_3_4 Output
- ; CHECK-DAG: [[x]] = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- ; CHECK-DAG: [[y]] = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- ; CHECK-DAG: [[z0]] = OpVariable %_ptr_Output_uint Output
- ; CHECK-DAG: [[z1]] = OpVariable %_ptr_Output_uint Output
- ; CHECK-DAG: [[w0]] = OpVariable %_ptr_Input_uint Input
- ; CHECK-DAG: [[w1]] = OpVariable %_ptr_Input_uint Input
- ; CHECK-DAG: [[u0]] = OpVariable %_ptr_Input__arr_uint_uint_3 Input
- ; CHECK-DAG: [[u1]] = OpVariable %_ptr_Input__arr_uint_uint_3 Input
- ; CHECK-DAG: [[v0]] = OpVariable %_ptr_Output__arr_uint_uint_4 Output
- ; CHECK-DAG: [[v1]] = OpVariable %_ptr_Output__arr_uint_uint_4 Output
- ; CHECK-DAG: [[v2]] = OpVariable %_ptr_Output__arr_uint_uint_4 Output
- ; CHECK-DAG: [[v3]] = OpVariable %_ptr_Output__arr_uint_uint_4 Output
- ; CHECK-DAG: [[v4]] = OpVariable %_ptr_Output__arr_uint_uint_4 Output
- ; CHECK-DAG: [[v5]] = OpVariable %_ptr_Output__arr_uint_uint_4 Output
- %void = OpTypeVoid
- %void_f = OpTypeFunction %void
- %func = OpFunction %void None %void_f
- %label = OpLabel
- ; CHECK: [[w0_value:%\w+]] = OpLoad %uint [[w0]]
- ; CHECK: [[w1_value:%\w+]] = OpLoad %uint [[w1]]
- ; CHECK: [[w_value:%\w+]] = OpCompositeConstruct %_arr_uint_uint_2 [[w0_value]] [[w1_value]]
- ; CHECK: [[w0:%\w+]] = OpCompositeExtract %uint [[w_value]] 0
- ; CHECK: OpStore [[z0]] [[w0]]
- ; CHECK: [[w1:%\w+]] = OpCompositeExtract %uint [[w_value]] 1
- ; CHECK: OpStore [[z1]] [[w1]]
- %w_value = OpLoad %_arr_uint_uint_2 %w
- OpStore %z %w_value
- ; CHECK: [[u00_ptr:%\w+]] = OpAccessChain %_ptr_Input_uint [[u0]] %uint_0
- ; CHECK: [[u00:%\w+]] = OpLoad %uint [[u00_ptr]]
- ; CHECK: [[u10_ptr:%\w+]] = OpAccessChain %_ptr_Input_uint [[u1]] %uint_0
- ; CHECK: [[u10:%\w+]] = OpLoad %uint [[u10_ptr]]
- ; CHECK: [[u01_ptr:%\w+]] = OpAccessChain %_ptr_Input_uint [[u0]] %uint_1
- ; CHECK: [[u01:%\w+]] = OpLoad %uint [[u01_ptr]]
- ; CHECK: [[u11_ptr:%\w+]] = OpAccessChain %_ptr_Input_uint [[u1]] %uint_1
- ; CHECK: [[u11:%\w+]] = OpLoad %uint [[u11_ptr]]
- ; CHECK: [[u02_ptr:%\w+]] = OpAccessChain %_ptr_Input_uint [[u0]] %uint_2
- ; CHECK: [[u02:%\w+]] = OpLoad %uint [[u02_ptr]]
- ; CHECK: [[u12_ptr:%\w+]] = OpAccessChain %_ptr_Input_uint [[u1]] %uint_2
- ; CHECK: [[u12:%\w+]] = OpLoad %uint [[u12_ptr]]
- ; CHECK-DAG: [[u0_val:%\w+]] = OpCompositeConstruct %_arr_uint_uint_2 [[u00]] [[u10]]
- ; CHECK-DAG: [[u1_val:%\w+]] = OpCompositeConstruct %_arr_uint_uint_2 [[u01]] [[u11]]
- ; CHECK-DAG: [[u2_val:%\w+]] = OpCompositeConstruct %_arr_uint_uint_2 [[u02]] [[u12]]
- ; CHECK: [[u_val:%\w+]] = OpCompositeConstruct %_arr__arr_uint_uint_2_uint_3 [[u0_val]] [[u1_val]] [[u2_val]]
- ; CHECK: [[ptr:%\w+]] = OpAccessChain %_ptr_Output_uint [[v0]] %uint_1
- ; CHECK: [[val:%\w+]] = OpCompositeExtract %uint [[u_val]] 0 0
- ; CHECK: OpStore [[ptr]] [[val]]
- ; CHECK: [[ptr:%\w+]] = OpAccessChain %_ptr_Output_uint [[v1]] %uint_1
- ; CHECK: [[val:%\w+]] = OpCompositeExtract %uint [[u_val]] 0 1
- ; CHECK: OpStore [[ptr]] [[val]]
- ; CHECK: [[ptr:%\w+]] = OpAccessChain %_ptr_Output_uint [[v2]] %uint_1
- ; CHECK: [[val:%\w+]] = OpCompositeExtract %uint [[u_val]] 1 0
- ; CHECK: OpStore [[ptr]] [[val]]
- ; CHECK: [[ptr:%\w+]] = OpAccessChain %_ptr_Output_uint [[v3]] %uint_1
- ; CHECK: [[val:%\w+]] = OpCompositeExtract %uint [[u_val]] 1 1
- ; CHECK: OpStore [[ptr]] [[val]]
- ; CHECK: [[ptr:%\w+]] = OpAccessChain %_ptr_Output_uint [[v4]] %uint_1
- ; CHECK: [[val:%\w+]] = OpCompositeExtract %uint [[u_val]] 2 0
- ; CHECK: OpStore [[ptr]] [[val]]
- ; CHECK: [[ptr:%\w+]] = OpAccessChain %_ptr_Output_uint [[v5]] %uint_1
- ; CHECK: [[val:%\w+]] = OpCompositeExtract %uint [[u_val]] 2 1
- ; CHECK: OpStore [[ptr]] [[val]]
- %v_ptr = OpAccessChain %_ptr_Output__arr_arr_uint_uint_2_3 %v %uint_1
- %u_val = OpLoad %_arr_arr_uint_uint_2_3 %u
- OpStore %v_ptr %u_val
- OpReturn
- OpFunctionEnd
- )";
- SinglePassRunAndMatch<InterfaceVariableScalarReplacement>(spirv, true);
- }
- TEST_F(InterfaceVariableScalarReplacementTest,
- CheckPatchDecorationPreservation) {
- // Make sure scalars for the variables with the extra arrayness have the extra
- // arrayness after running the pass while others do not have it.
- // Only "y" does not have the extra arrayness in the following SPIR-V.
- const std::string spirv = R"(
- OpCapability Shader
- OpCapability Tessellation
- OpMemoryModel Logical GLSL450
- OpEntryPoint TessellationEvaluation %func "shader" %x %y %z %w
- OpDecorate %z Patch
- OpDecorate %w Patch
- OpDecorate %z Location 0
- OpDecorate %x Location 2
- OpDecorate %y Location 0
- OpDecorate %w Location 1
- OpName %x "x"
- OpName %y "y"
- OpName %z "z"
- OpName %w "w"
- ; CHECK: OpName [[y:%\w+]] "y"
- ; CHECK-NOT: OpName {{%\w+}} "y"
- ; CHECK-DAG: OpName [[z0:%\w+]] "z"
- ; CHECK-DAG: OpName [[z1:%\w+]] "z"
- ; CHECK-DAG: OpName [[w0:%\w+]] "w"
- ; CHECK-DAG: OpName [[w1:%\w+]] "w"
- ; CHECK-DAG: OpName [[x0:%\w+]] "x"
- ; CHECK-DAG: OpName [[x1:%\w+]] "x"
- %uint = OpTypeInt 32 0
- %uint_2 = OpConstant %uint 2
- %_arr_uint_uint_2 = OpTypeArray %uint %uint_2
- %_ptr_Output__arr_uint_uint_2 = OpTypePointer Output %_arr_uint_uint_2
- %_ptr_Input__arr_uint_uint_2 = OpTypePointer Input %_arr_uint_uint_2
- %z = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %x = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %y = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- %w = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- ; CHECK-DAG: [[y]] = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- ; CHECK-DAG: [[z0]] = OpVariable %_ptr_Output_uint Output
- ; CHECK-DAG: [[z1]] = OpVariable %_ptr_Output_uint Output
- ; CHECK-DAG: [[w0]] = OpVariable %_ptr_Input_uint Input
- ; CHECK-DAG: [[w1]] = OpVariable %_ptr_Input_uint Input
- ; CHECK-DAG: [[x0]] = OpVariable %_ptr_Output_uint Output
- ; CHECK-DAG: [[x1]] = OpVariable %_ptr_Output_uint Output
- %void = OpTypeVoid
- %void_f = OpTypeFunction %void
- %func = OpFunction %void None %void_f
- %label = OpLabel
- OpReturn
- OpFunctionEnd
- )";
- SinglePassRunAndMatch<InterfaceVariableScalarReplacement>(spirv, true);
- }
- TEST_F(InterfaceVariableScalarReplacementTest,
- CheckEntryPointInterfaceOperands) {
- const std::string spirv = R"(
- OpCapability Shader
- OpCapability Tessellation
- OpMemoryModel Logical GLSL450
- OpEntryPoint TessellationEvaluation %tess "tess" %x %y
- OpEntryPoint Vertex %vert "vert" %w
- OpDecorate %z Location 0
- OpDecorate %x Location 2
- OpDecorate %y Location 0
- OpDecorate %w Location 1
- OpName %x "x"
- OpName %y "y"
- OpName %z "z"
- OpName %w "w"
- ; CHECK: OpName [[y:%\w+]] "y"
- ; CHECK-NOT: OpName {{%\w+}} "y"
- ; CHECK-DAG: OpName [[x0:%\w+]] "x"
- ; CHECK-DAG: OpName [[x1:%\w+]] "x"
- ; CHECK-DAG: OpName [[w0:%\w+]] "w"
- ; CHECK-DAG: OpName [[w1:%\w+]] "w"
- ; CHECK-DAG: OpName [[z:%\w+]] "z"
- ; CHECK-NOT: OpName {{%\w+}} "z"
- %uint = OpTypeInt 32 0
- %uint_2 = OpConstant %uint 2
- %_arr_uint_uint_2 = OpTypeArray %uint %uint_2
- %_ptr_Output__arr_uint_uint_2 = OpTypePointer Output %_arr_uint_uint_2
- %_ptr_Input__arr_uint_uint_2 = OpTypePointer Input %_arr_uint_uint_2
- %z = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %x = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %y = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- %w = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- ; CHECK-DAG: [[y]] = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- ; CHECK-DAG: [[z]] = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- ; CHECK-DAG: [[w0]] = OpVariable %_ptr_Input_uint Input
- ; CHECK-DAG: [[w1]] = OpVariable %_ptr_Input_uint Input
- ; CHECK-DAG: [[x0]] = OpVariable %_ptr_Output_uint Output
- ; CHECK-DAG: [[x1]] = OpVariable %_ptr_Output_uint Output
- %void = OpTypeVoid
- %void_f = OpTypeFunction %void
- %tess = OpFunction %void None %void_f
- %bb0 = OpLabel
- OpReturn
- OpFunctionEnd
- %vert = OpFunction %void None %void_f
- %bb1 = OpLabel
- OpReturn
- OpFunctionEnd
- )";
- SinglePassRunAndMatch<InterfaceVariableScalarReplacement>(spirv, true);
- }
- class InterfaceVarSROAErrorTest : public PassTest<::testing::Test> {
- public:
- InterfaceVarSROAErrorTest()
- : consumer_([this](spv_message_level_t level, const char*,
- const spv_position_t& position, const char* message) {
- if (!error_message_.empty()) error_message_ += "\n";
- switch (level) {
- case SPV_MSG_FATAL:
- case SPV_MSG_INTERNAL_ERROR:
- case SPV_MSG_ERROR:
- error_message_ += "ERROR";
- break;
- case SPV_MSG_WARNING:
- error_message_ += "WARNING";
- break;
- case SPV_MSG_INFO:
- error_message_ += "INFO";
- break;
- case SPV_MSG_DEBUG:
- error_message_ += "DEBUG";
- break;
- }
- error_message_ +=
- ": " + std::to_string(position.index) + ": " + message;
- }) {}
- Pass::Status RunPass(const std::string& text) {
- std::unique_ptr<IRContext> context_ =
- spvtools::BuildModule(SPV_ENV_UNIVERSAL_1_2, consumer_, text);
- if (!context_.get()) return Pass::Status::Failure;
- PassManager manager;
- manager.SetMessageConsumer(consumer_);
- manager.AddPass<InterfaceVariableScalarReplacement>();
- return manager.Run(context_.get());
- }
- std::string GetErrorMessage() const { return error_message_; }
- void TearDown() override { error_message_.clear(); }
- private:
- spvtools::MessageConsumer consumer_;
- std::string error_message_;
- };
- TEST_F(InterfaceVarSROAErrorTest, CheckConflictOfExtraArraynessBetweenEntries) {
- const std::string spirv = R"(
- OpCapability Shader
- OpCapability Tessellation
- OpMemoryModel Logical GLSL450
- OpEntryPoint TessellationControl %tess "tess" %x %y %z
- OpEntryPoint Vertex %vert "vert" %z %w
- OpDecorate %z Location 0
- OpDecorate %x Location 2
- OpDecorate %y Location 0
- OpDecorate %w Location 1
- OpName %x "x"
- OpName %y "y"
- OpName %z "z"
- OpName %w "w"
- %uint = OpTypeInt 32 0
- %uint_2 = OpConstant %uint 2
- %_arr_uint_uint_2 = OpTypeArray %uint %uint_2
- %_ptr_Output__arr_uint_uint_2 = OpTypePointer Output %_arr_uint_uint_2
- %_ptr_Input__arr_uint_uint_2 = OpTypePointer Input %_arr_uint_uint_2
- %z = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %x = OpVariable %_ptr_Output__arr_uint_uint_2 Output
- %y = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- %w = OpVariable %_ptr_Input__arr_uint_uint_2 Input
- %void = OpTypeVoid
- %void_f = OpTypeFunction %void
- %tess = OpFunction %void None %void_f
- %bb0 = OpLabel
- OpReturn
- OpFunctionEnd
- %vert = OpFunction %void None %void_f
- %bb1 = OpLabel
- OpReturn
- OpFunctionEnd
- )";
- EXPECT_EQ(RunPass(spirv), Pass::Status::Failure);
- const char expected_error[] =
- "ERROR: 0: A variable is arrayed for an entry point but it is not "
- "arrayed for another entry point\n"
- " %z = OpVariable %_ptr_Output__arr_uint_uint_2 Output";
- EXPECT_STREQ(GetErrorMessage().c_str(), expected_error);
- }
- } // namespace
- } // namespace opt
- } // namespace spvtools
|