Skip to content

Commit 780c653

Browse files
spencer-lunargjeremyg-lunarg
authored andcommitted
layers: Remove GPL StageCount VU
1 parent ef8339e commit 780c653

File tree

3 files changed

+11
-29
lines changed

3 files changed

+11
-29
lines changed

layers/stateless/sl_pipeline.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
/* Copyright (c) 2015-2023 The Khronos Group Inc.
2-
* Copyright (c) 2015-2023 Valve Corporation
3-
* Copyright (c) 2015-2023 LunarG, Inc.
4-
* Copyright (C) 2015-2023 Google Inc.
1+
/* Copyright (c) 2015-2024 The Khronos Group Inc.
2+
* Copyright (c) 2015-2024 Valve Corporation
3+
* Copyright (c) 2015-2024 LunarG, Inc.
4+
* Copyright (C) 2015-2024 Google Inc.
55
*
66
* Licensed under the Apache License, Version 2.0 (the "License");
77
* you may not use this file except in compliance with the License.
@@ -292,9 +292,9 @@ bool StatelessValidation::manual_PreCallValidateCreateGraphicsPipelines(
292292

293293
if (graphics_lib_info && (graphics_lib_info->flags & (VK_GRAPHICS_PIPELINE_LIBRARY_PRE_RASTERIZATION_SHADERS_BIT_EXT |
294294
VK_GRAPHICS_PIPELINE_LIBRARY_FRAGMENT_SHADER_BIT_EXT))) {
295-
skip |= ValidateArray(create_info_loc.dot(Field::stageCount), create_info_loc.dot(Field::pStages),
296-
create_info.stageCount, &create_info.pStages, true, true,
297-
"VUID-VkGraphicsPipelineCreateInfo-flags-06644", "VUID-VkGraphicsPipelineCreateInfo-flags-06640");
295+
skip |=
296+
ValidateArray(create_info_loc.dot(Field::stageCount), create_info_loc.dot(Field::pStages), create_info.stageCount,
297+
&create_info.pStages, false, true, kVUIDUndefined, "VUID-VkGraphicsPipelineCreateInfo-flags-06640");
298298
}
299299

300300
// <VkDynamicState, index in pDynamicStates, hash for enum key>

tests/unit/gpu_av_oob.cpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
/*
2-
* Copyright (c) 2020-2023 The Khronos Group Inc.
3-
* Copyright (c) 2020-2023 Valve Corporation
4-
* Copyright (c) 2020-2023 LunarG, Inc.
5-
* Copyright (c) 2020-2023 Google, Inc.
2+
* Copyright (c) 2020-2024 The Khronos Group Inc.
3+
* Copyright (c) 2020-2024 Valve Corporation
4+
* Copyright (c) 2020-2024 LunarG, Inc.
5+
* Copyright (c) 2020-2024 Google, Inc.
66
*
77
* Licensed under the Apache License, Version 2.0 (the "License");
88
* you may not use this file except in compliance with the License.
@@ -563,7 +563,6 @@ TEST_F(NegativeGpuAVOOB, GPL) {
563563
const auto render_pass = pre_raster.gp_ci_.renderPass;
564564
const auto subpass = pre_raster.gp_ci_.subpass;
565565

566-
m_errorMonitor->SetAllowedFailureMsg("VUID-VkGraphicsPipelineCreateInfo-flags-06644");
567566
CreatePipelineHelper fragment(*this);
568567
fragment.InitFragmentLibInfo(nullptr);
569568
fragment.gp_ci_.stageCount = 0;

tests/unit/graphics_library.cpp

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -869,22 +869,6 @@ TEST_F(NegativeGraphicsLibrary, FragmentStateWithPreRaster) {
869869
m_errorMonitor->VerifyFound();
870870
}
871871

872-
TEST_F(NegativeGraphicsLibrary, StageCount) {
873-
RETURN_IF_SKIP(InitBasicGraphicsLibrary());
874-
InitRenderTarget();
875-
876-
CreatePipelineHelper pre_raster_lib(*this);
877-
const auto vs_spv = GLSLToSPV(VK_SHADER_STAGE_VERTEX_BIT, kVertexMinimalGlsl);
878-
vkt::GraphicsPipelineLibraryStage vs_stage(vs_spv, VK_SHADER_STAGE_VERTEX_BIT);
879-
pre_raster_lib.InitPreRasterLibInfo(&vs_stage.stage_ci);
880-
pre_raster_lib.InitState();
881-
pre_raster_lib.gp_ci_.stageCount = 0;
882-
pre_raster_lib.gp_ci_.layout = pre_raster_lib.pipeline_layout_.handle();
883-
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-VkGraphicsPipelineCreateInfo-flags-06644");
884-
pre_raster_lib.CreateGraphicsPipeline(false);
885-
m_errorMonitor->VerifyFound();
886-
}
887-
888872
TEST_F(NegativeGraphicsLibrary, NullStages) {
889873
RETURN_IF_SKIP(InitBasicGraphicsLibrary());
890874
InitRenderTarget();
@@ -910,7 +894,6 @@ TEST_F(NegativeGraphicsLibrary, MissingShaderStages) {
910894
pipe.InitState();
911895

912896
// set in stateless
913-
m_errorMonitor->SetAllowedFailureMsg("VUID-VkGraphicsPipelineCreateInfo-flags-06644");
914897
// 02096 is effectively unrelated, but gets triggered due to lack of mesh shader extension
915898
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-VkGraphicsPipelineCreateInfo-stage-02096");
916899
m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-VkGraphicsPipelineCreateInfo-pStages-06896");

0 commit comments

Comments
 (0)