blob: 83b649c6c223db065e3e504ae22cd1c81735a7b1 [file] [log] [blame]
Karl Schultz7b024b42018-08-30 16:18:18 -06001/* Copyright (c) 2015-2019 The Khronos Group Inc.
2 * Copyright (c) 2015-2019 Valve Corporation
3 * Copyright (c) 2015-2019 LunarG, Inc.
4 * Copyright (C) 2015-2019 Google Inc.
Chris Forbes47567b72017-06-09 12:09:45 -07005 *
6 * Licensed under the Apache License, Version 2.0 (the "License");
7 * you may not use this file except in compliance with the License.
8 * You may obtain a copy of the License at
9 *
10 * http://www.apache.org/licenses/LICENSE-2.0
11 *
12 * Unless required by applicable law or agreed to in writing, software
13 * distributed under the License is distributed on an "AS IS" BASIS,
14 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
15 * See the License for the specific language governing permissions and
16 * limitations under the License.
17 *
18 * Author: Chris Forbes <chrisf@ijw.co.nz>
Dave Houlton51653902018-06-22 17:32:13 -060019 * Author: Dave Houlton <daveh@lunarg.com>
Chris Forbes47567b72017-06-09 12:09:45 -070020 */
21
22#include <cinttypes>
23#include <cassert>
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +020024#include <chrono>
Chris Forbes47567b72017-06-09 12:09:45 -070025#include <vector>
26#include <unordered_map>
27#include <string>
28#include <sstream>
29#include <SPIRV/spirv.hpp>
30#include "vk_loader_platform.h"
31#include "vk_enum_string_helper.h"
Chris Forbes47567b72017-06-09 12:09:45 -070032#include "vk_layer_data.h"
33#include "vk_layer_extension_utils.h"
34#include "vk_layer_utils.h"
Mark Lobodzinskib56bbb92019-02-18 11:49:59 -070035#include "chassis.h"
Chris Forbes47567b72017-06-09 12:09:45 -070036#include "core_validation.h"
Chris Forbes47567b72017-06-09 12:09:45 -070037#include "shader_validation.h"
Chris Forbes4ae55b32017-06-09 14:42:56 -070038#include "spirv-tools/libspirv.h"
Chris Forbes9a61e082017-07-24 15:35:29 -070039#include "xxhash.h"
Chris Forbes47567b72017-06-09 12:09:45 -070040
41enum FORMAT_TYPE {
42 FORMAT_TYPE_FLOAT = 1, // UNORM, SNORM, FLOAT, USCALED, SSCALED, SRGB -- anything we consider float in the shader
43 FORMAT_TYPE_SINT = 2,
44 FORMAT_TYPE_UINT = 4,
45};
46
47typedef std::pair<unsigned, unsigned> location_t;
48
49struct interface_var {
50 uint32_t id;
51 uint32_t type_id;
52 uint32_t offset;
53 bool is_patch;
54 bool is_block_member;
55 bool is_relaxed_precision;
56 // TODO: collect the name, too? Isn't required to be present.
57};
58
59struct shader_stage_attributes {
60 char const *const name;
61 bool arrayed_input;
62 bool arrayed_output;
Ari Suonpaa696b3432019-03-11 14:02:57 +020063 VkShaderStageFlags stage;
Chris Forbes47567b72017-06-09 12:09:45 -070064};
65
66static shader_stage_attributes shader_stage_attribs[] = {
Ari Suonpaa696b3432019-03-11 14:02:57 +020067 {"vertex shader", false, false, VK_SHADER_STAGE_VERTEX_BIT},
68 {"tessellation control shader", true, true, VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT},
69 {"tessellation evaluation shader", true, false, VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT},
70 {"geometry shader", true, false, VK_SHADER_STAGE_GEOMETRY_BIT},
71 {"fragment shader", false, false, VK_SHADER_STAGE_FRAGMENT_BIT},
Chris Forbes47567b72017-06-09 12:09:45 -070072};
73
74// SPIRV utility functions
Mark Lobodzinski3c59d972019-04-25 11:28:14 -060075void SHADER_MODULE_STATE::BuildDefIndex() {
Chris Forbes47567b72017-06-09 12:09:45 -070076 for (auto insn : *this) {
77 switch (insn.opcode()) {
78 // Types
79 case spv::OpTypeVoid:
80 case spv::OpTypeBool:
81 case spv::OpTypeInt:
82 case spv::OpTypeFloat:
83 case spv::OpTypeVector:
84 case spv::OpTypeMatrix:
85 case spv::OpTypeImage:
86 case spv::OpTypeSampler:
87 case spv::OpTypeSampledImage:
88 case spv::OpTypeArray:
89 case spv::OpTypeRuntimeArray:
90 case spv::OpTypeStruct:
91 case spv::OpTypeOpaque:
92 case spv::OpTypePointer:
93 case spv::OpTypeFunction:
94 case spv::OpTypeEvent:
95 case spv::OpTypeDeviceEvent:
96 case spv::OpTypeReserveId:
97 case spv::OpTypeQueue:
98 case spv::OpTypePipe:
Shannon McPherson0fa28232018-11-01 11:59:02 -060099 case spv::OpTypeAccelerationStructureNV:
Jeff Bolze4356752019-03-07 11:23:46 -0600100 case spv::OpTypeCooperativeMatrixNV:
Chris Forbes47567b72017-06-09 12:09:45 -0700101 def_index[insn.word(1)] = insn.offset();
102 break;
103
104 // Fixed constants
105 case spv::OpConstantTrue:
106 case spv::OpConstantFalse:
107 case spv::OpConstant:
108 case spv::OpConstantComposite:
109 case spv::OpConstantSampler:
110 case spv::OpConstantNull:
111 def_index[insn.word(2)] = insn.offset();
112 break;
113
114 // Specialization constants
115 case spv::OpSpecConstantTrue:
116 case spv::OpSpecConstantFalse:
117 case spv::OpSpecConstant:
118 case spv::OpSpecConstantComposite:
119 case spv::OpSpecConstantOp:
120 def_index[insn.word(2)] = insn.offset();
121 break;
122
123 // Variables
124 case spv::OpVariable:
125 def_index[insn.word(2)] = insn.offset();
126 break;
127
128 // Functions
129 case spv::OpFunction:
130 def_index[insn.word(2)] = insn.offset();
131 break;
132
133 default:
134 // We don't care about any other defs for now.
135 break;
136 }
137 }
138}
139
Jeff Bolz105d6492018-09-29 15:46:44 -0500140unsigned ExecutionModelToShaderStageFlagBits(unsigned mode) {
141 switch (mode) {
142 case spv::ExecutionModelVertex:
143 return VK_SHADER_STAGE_VERTEX_BIT;
144 case spv::ExecutionModelTessellationControl:
145 return VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT;
146 case spv::ExecutionModelTessellationEvaluation:
147 return VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT;
148 case spv::ExecutionModelGeometry:
149 return VK_SHADER_STAGE_GEOMETRY_BIT;
150 case spv::ExecutionModelFragment:
151 return VK_SHADER_STAGE_FRAGMENT_BIT;
152 case spv::ExecutionModelGLCompute:
153 return VK_SHADER_STAGE_COMPUTE_BIT;
Shannon McPherson0fa28232018-11-01 11:59:02 -0600154 case spv::ExecutionModelRayGenerationNV:
Eric Werness30127fd2018-10-31 21:01:03 -0700155 return VK_SHADER_STAGE_RAYGEN_BIT_NV;
Shannon McPherson0fa28232018-11-01 11:59:02 -0600156 case spv::ExecutionModelAnyHitNV:
Eric Werness30127fd2018-10-31 21:01:03 -0700157 return VK_SHADER_STAGE_ANY_HIT_BIT_NV;
Shannon McPherson0fa28232018-11-01 11:59:02 -0600158 case spv::ExecutionModelClosestHitNV:
Eric Werness30127fd2018-10-31 21:01:03 -0700159 return VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV;
Shannon McPherson0fa28232018-11-01 11:59:02 -0600160 case spv::ExecutionModelMissNV:
Eric Werness30127fd2018-10-31 21:01:03 -0700161 return VK_SHADER_STAGE_MISS_BIT_NV;
Shannon McPherson0fa28232018-11-01 11:59:02 -0600162 case spv::ExecutionModelIntersectionNV:
Eric Werness30127fd2018-10-31 21:01:03 -0700163 return VK_SHADER_STAGE_INTERSECTION_BIT_NV;
Shannon McPherson0fa28232018-11-01 11:59:02 -0600164 case spv::ExecutionModelCallableNV:
Eric Werness30127fd2018-10-31 21:01:03 -0700165 return VK_SHADER_STAGE_CALLABLE_BIT_NV;
Jeff Bolz105d6492018-09-29 15:46:44 -0500166 case spv::ExecutionModelTaskNV:
167 return VK_SHADER_STAGE_TASK_BIT_NV;
168 case spv::ExecutionModelMeshNV:
169 return VK_SHADER_STAGE_MESH_BIT_NV;
170 default:
171 return 0;
172 }
173}
174
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600175static spirv_inst_iter FindEntrypoint(SHADER_MODULE_STATE const *src, char const *name, VkShaderStageFlagBits stageBits) {
Chris Forbes47567b72017-06-09 12:09:45 -0700176 for (auto insn : *src) {
177 if (insn.opcode() == spv::OpEntryPoint) {
178 auto entrypointName = (char const *)&insn.word(3);
Jeff Bolz105d6492018-09-29 15:46:44 -0500179 auto executionModel = insn.word(1);
180 auto entrypointStageBits = ExecutionModelToShaderStageFlagBits(executionModel);
Chris Forbes47567b72017-06-09 12:09:45 -0700181
182 if (!strcmp(entrypointName, name) && (entrypointStageBits & stageBits)) {
183 return insn;
184 }
185 }
186 }
187
188 return src->end();
189}
190
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600191static char const *StorageClassName(unsigned sc) {
Chris Forbes47567b72017-06-09 12:09:45 -0700192 switch (sc) {
193 case spv::StorageClassInput:
194 return "input";
195 case spv::StorageClassOutput:
196 return "output";
197 case spv::StorageClassUniformConstant:
198 return "const uniform";
199 case spv::StorageClassUniform:
200 return "uniform";
201 case spv::StorageClassWorkgroup:
202 return "workgroup local";
203 case spv::StorageClassCrossWorkgroup:
204 return "workgroup global";
205 case spv::StorageClassPrivate:
206 return "private global";
207 case spv::StorageClassFunction:
208 return "function";
209 case spv::StorageClassGeneric:
210 return "generic";
211 case spv::StorageClassAtomicCounter:
212 return "atomic counter";
213 case spv::StorageClassImage:
214 return "image";
215 case spv::StorageClassPushConstant:
216 return "push constant";
Chris Forbes9f89d752018-03-07 12:57:48 -0800217 case spv::StorageClassStorageBuffer:
218 return "storage buffer";
Chris Forbes47567b72017-06-09 12:09:45 -0700219 default:
220 return "unknown";
221 }
222}
223
224// Get the value of an integral constant
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600225unsigned GetConstantValue(SHADER_MODULE_STATE const *src, unsigned id) {
Chris Forbes47567b72017-06-09 12:09:45 -0700226 auto value = src->get_def(id);
227 assert(value != src->end());
228
229 if (value.opcode() != spv::OpConstant) {
230 // TODO: Either ensure that the specialization transform is already performed on a module we're
231 // considering here, OR -- specialize on the fly now.
232 return 1;
233 }
234
235 return value.word(3);
236}
237
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600238static void DescribeTypeInner(std::ostringstream &ss, SHADER_MODULE_STATE const *src, unsigned type) {
Chris Forbes47567b72017-06-09 12:09:45 -0700239 auto insn = src->get_def(type);
240 assert(insn != src->end());
241
242 switch (insn.opcode()) {
243 case spv::OpTypeBool:
244 ss << "bool";
245 break;
246 case spv::OpTypeInt:
247 ss << (insn.word(3) ? 's' : 'u') << "int" << insn.word(2);
248 break;
249 case spv::OpTypeFloat:
250 ss << "float" << insn.word(2);
251 break;
252 case spv::OpTypeVector:
253 ss << "vec" << insn.word(3) << " of ";
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600254 DescribeTypeInner(ss, src, insn.word(2));
Chris Forbes47567b72017-06-09 12:09:45 -0700255 break;
256 case spv::OpTypeMatrix:
257 ss << "mat" << insn.word(3) << " of ";
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600258 DescribeTypeInner(ss, src, insn.word(2));
Chris Forbes47567b72017-06-09 12:09:45 -0700259 break;
260 case spv::OpTypeArray:
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600261 ss << "arr[" << GetConstantValue(src, insn.word(3)) << "] of ";
262 DescribeTypeInner(ss, src, insn.word(2));
Chris Forbes47567b72017-06-09 12:09:45 -0700263 break;
Chris Forbes062f1222018-08-21 15:34:15 -0700264 case spv::OpTypeRuntimeArray:
265 ss << "runtime arr[] of ";
266 DescribeTypeInner(ss, src, insn.word(2));
267 break;
Chris Forbes47567b72017-06-09 12:09:45 -0700268 case spv::OpTypePointer:
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600269 ss << "ptr to " << StorageClassName(insn.word(2)) << " ";
270 DescribeTypeInner(ss, src, insn.word(3));
Chris Forbes47567b72017-06-09 12:09:45 -0700271 break;
272 case spv::OpTypeStruct: {
273 ss << "struct of (";
274 for (unsigned i = 2; i < insn.len(); i++) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600275 DescribeTypeInner(ss, src, insn.word(i));
Chris Forbes47567b72017-06-09 12:09:45 -0700276 if (i == insn.len() - 1) {
277 ss << ")";
278 } else {
279 ss << ", ";
280 }
281 }
282 break;
283 }
284 case spv::OpTypeSampler:
285 ss << "sampler";
286 break;
287 case spv::OpTypeSampledImage:
288 ss << "sampler+";
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600289 DescribeTypeInner(ss, src, insn.word(2));
Chris Forbes47567b72017-06-09 12:09:45 -0700290 break;
291 case spv::OpTypeImage:
292 ss << "image(dim=" << insn.word(3) << ", sampled=" << insn.word(7) << ")";
293 break;
Shannon McPherson0fa28232018-11-01 11:59:02 -0600294 case spv::OpTypeAccelerationStructureNV:
Jeff Bolz105d6492018-09-29 15:46:44 -0500295 ss << "accelerationStruture";
296 break;
Chris Forbes47567b72017-06-09 12:09:45 -0700297 default:
298 ss << "oddtype";
299 break;
300 }
301}
302
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600303static std::string DescribeType(SHADER_MODULE_STATE const *src, unsigned type) {
Chris Forbes47567b72017-06-09 12:09:45 -0700304 std::ostringstream ss;
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600305 DescribeTypeInner(ss, src, type);
Chris Forbes47567b72017-06-09 12:09:45 -0700306 return ss.str();
307}
308
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600309static bool IsNarrowNumericType(spirv_inst_iter type) {
Chris Forbes47567b72017-06-09 12:09:45 -0700310 if (type.opcode() != spv::OpTypeInt && type.opcode() != spv::OpTypeFloat) return false;
311 return type.word(2) < 64;
312}
313
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600314static bool TypesMatch(SHADER_MODULE_STATE const *a, SHADER_MODULE_STATE const *b, unsigned a_type, unsigned b_type, bool a_arrayed,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600315 bool b_arrayed, bool relaxed) {
Chris Forbes47567b72017-06-09 12:09:45 -0700316 // Walk two type trees together, and complain about differences
317 auto a_insn = a->get_def(a_type);
318 auto b_insn = b->get_def(b_type);
319 assert(a_insn != a->end());
320 assert(b_insn != b->end());
321
Chris Forbes062f1222018-08-21 15:34:15 -0700322 // Ignore runtime-sized arrays-- they cannot appear in these interfaces.
323
Chris Forbes47567b72017-06-09 12:09:45 -0700324 if (a_arrayed && a_insn.opcode() == spv::OpTypeArray) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600325 return TypesMatch(a, b, a_insn.word(2), b_type, false, b_arrayed, relaxed);
Chris Forbes47567b72017-06-09 12:09:45 -0700326 }
327
328 if (b_arrayed && b_insn.opcode() == spv::OpTypeArray) {
329 // We probably just found the extra level of arrayness in b_type: compare the type inside it to a_type
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600330 return TypesMatch(a, b, a_type, b_insn.word(2), a_arrayed, false, relaxed);
Chris Forbes47567b72017-06-09 12:09:45 -0700331 }
332
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600333 if (a_insn.opcode() == spv::OpTypeVector && relaxed && IsNarrowNumericType(b_insn)) {
334 return TypesMatch(a, b, a_insn.word(2), b_type, a_arrayed, b_arrayed, false);
Chris Forbes47567b72017-06-09 12:09:45 -0700335 }
336
337 if (a_insn.opcode() != b_insn.opcode()) {
338 return false;
339 }
340
341 if (a_insn.opcode() == spv::OpTypePointer) {
342 // Match on pointee type. storage class is expected to differ
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600343 return TypesMatch(a, b, a_insn.word(3), b_insn.word(3), a_arrayed, b_arrayed, relaxed);
Chris Forbes47567b72017-06-09 12:09:45 -0700344 }
345
346 if (a_arrayed || b_arrayed) {
347 // If we havent resolved array-of-verts by here, we're not going to.
348 return false;
349 }
350
351 switch (a_insn.opcode()) {
352 case spv::OpTypeBool:
353 return true;
354 case spv::OpTypeInt:
355 // Match on width, signedness
356 return a_insn.word(2) == b_insn.word(2) && a_insn.word(3) == b_insn.word(3);
357 case spv::OpTypeFloat:
358 // Match on width
359 return a_insn.word(2) == b_insn.word(2);
360 case spv::OpTypeVector:
361 // Match on element type, count.
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600362 if (!TypesMatch(a, b, a_insn.word(2), b_insn.word(2), a_arrayed, b_arrayed, false)) return false;
363 if (relaxed && IsNarrowNumericType(a->get_def(a_insn.word(2)))) {
Chris Forbes47567b72017-06-09 12:09:45 -0700364 return a_insn.word(3) >= b_insn.word(3);
365 } else {
366 return a_insn.word(3) == b_insn.word(3);
367 }
368 case spv::OpTypeMatrix:
369 // Match on element type, count.
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600370 return TypesMatch(a, b, a_insn.word(2), b_insn.word(2), a_arrayed, b_arrayed, false) &&
Dave Houltona9df0ce2018-02-07 10:51:23 -0700371 a_insn.word(3) == b_insn.word(3);
Chris Forbes47567b72017-06-09 12:09:45 -0700372 case spv::OpTypeArray:
373 // Match on element type, count. these all have the same layout. we don't get here if b_arrayed. This differs from
374 // vector & matrix types in that the array size is the id of a constant instruction, * not a literal within OpTypeArray
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600375 return TypesMatch(a, b, a_insn.word(2), b_insn.word(2), a_arrayed, b_arrayed, false) &&
376 GetConstantValue(a, a_insn.word(3)) == GetConstantValue(b, b_insn.word(3));
Chris Forbes47567b72017-06-09 12:09:45 -0700377 case spv::OpTypeStruct:
378 // Match on all element types
Dave Houltona9df0ce2018-02-07 10:51:23 -0700379 {
380 if (a_insn.len() != b_insn.len()) {
381 return false; // Structs cannot match if member counts differ
Chris Forbes47567b72017-06-09 12:09:45 -0700382 }
Chris Forbes47567b72017-06-09 12:09:45 -0700383
Dave Houltona9df0ce2018-02-07 10:51:23 -0700384 for (unsigned i = 2; i < a_insn.len(); i++) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600385 if (!TypesMatch(a, b, a_insn.word(i), b_insn.word(i), a_arrayed, b_arrayed, false)) {
Dave Houltona9df0ce2018-02-07 10:51:23 -0700386 return false;
387 }
388 }
389
390 return true;
391 }
Chris Forbes47567b72017-06-09 12:09:45 -0700392 default:
393 // Remaining types are CLisms, or may not appear in the interfaces we are interested in. Just claim no match.
394 return false;
395 }
396}
397
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600398static unsigned ValueOrDefault(std::unordered_map<unsigned, unsigned> const &map, unsigned id, unsigned def) {
Chris Forbes47567b72017-06-09 12:09:45 -0700399 auto it = map.find(id);
400 if (it == map.end())
401 return def;
402 else
403 return it->second;
404}
405
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600406static unsigned GetLocationsConsumedByType(SHADER_MODULE_STATE const *src, unsigned type, bool strip_array_level) {
Chris Forbes47567b72017-06-09 12:09:45 -0700407 auto insn = src->get_def(type);
408 assert(insn != src->end());
409
410 switch (insn.opcode()) {
411 case spv::OpTypePointer:
412 // See through the ptr -- this is only ever at the toplevel for graphics shaders we're never actually passing
413 // pointers around.
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600414 return GetLocationsConsumedByType(src, insn.word(3), strip_array_level);
Chris Forbes47567b72017-06-09 12:09:45 -0700415 case spv::OpTypeArray:
416 if (strip_array_level) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600417 return GetLocationsConsumedByType(src, insn.word(2), false);
Chris Forbes47567b72017-06-09 12:09:45 -0700418 } else {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600419 return GetConstantValue(src, insn.word(3)) * GetLocationsConsumedByType(src, insn.word(2), false);
Chris Forbes47567b72017-06-09 12:09:45 -0700420 }
421 case spv::OpTypeMatrix:
422 // Num locations is the dimension * element size
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600423 return insn.word(3) * GetLocationsConsumedByType(src, insn.word(2), false);
Chris Forbes47567b72017-06-09 12:09:45 -0700424 case spv::OpTypeVector: {
425 auto scalar_type = src->get_def(insn.word(2));
426 auto bit_width =
427 (scalar_type.opcode() == spv::OpTypeInt || scalar_type.opcode() == spv::OpTypeFloat) ? scalar_type.word(2) : 32;
428
429 // Locations are 128-bit wide; 3- and 4-component vectors of 64 bit types require two.
430 return (bit_width * insn.word(3) + 127) / 128;
431 }
432 default:
433 // Everything else is just 1.
434 return 1;
435
436 // TODO: extend to handle 64bit scalar types, whose vectors may need multiple locations.
437 }
438}
439
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600440static unsigned GetComponentsConsumedByType(SHADER_MODULE_STATE const *src, unsigned type, bool strip_array_level) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200441 auto insn = src->get_def(type);
442 assert(insn != src->end());
443
444 switch (insn.opcode()) {
445 case spv::OpTypePointer:
446 // See through the ptr -- this is only ever at the toplevel for graphics shaders we're never actually passing
447 // pointers around.
448 return GetComponentsConsumedByType(src, insn.word(3), strip_array_level);
449 case spv::OpTypeStruct: {
450 uint32_t sum = 0;
451 for (uint32_t i = 2; i < insn.len(); i++) { // i=2 to skip word(0) and word(1)=ID of struct
452 sum += GetComponentsConsumedByType(src, insn.word(i), false);
453 }
454 return sum;
455 }
Jeff Bolze9ee3d82019-05-29 13:45:13 -0500456 case spv::OpTypeArray:
457 if (strip_array_level) {
458 return GetComponentsConsumedByType(src, insn.word(2), false);
459 } else {
460 return GetConstantValue(src, insn.word(3)) * GetComponentsConsumedByType(src, insn.word(2), false);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200461 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +0200462 case spv::OpTypeMatrix:
463 // Num locations is the dimension * element size
464 return insn.word(3) * GetComponentsConsumedByType(src, insn.word(2), false);
465 case spv::OpTypeVector: {
466 auto scalar_type = src->get_def(insn.word(2));
467 auto bit_width =
468 (scalar_type.opcode() == spv::OpTypeInt || scalar_type.opcode() == spv::OpTypeFloat) ? scalar_type.word(2) : 32;
469 // One component is 32-bit
470 return (bit_width * insn.word(3) + 31) / 32;
471 }
472 case spv::OpTypeFloat: {
473 auto bit_width = insn.word(2);
474 return (bit_width + 31) / 32;
475 }
476 case spv::OpTypeInt: {
477 auto bit_width = insn.word(2);
478 return (bit_width + 31) / 32;
479 }
480 case spv::OpConstant:
481 return GetComponentsConsumedByType(src, insn.word(1), false);
482 default:
483 return 0;
484 }
485}
486
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600487static unsigned GetLocationsConsumedByFormat(VkFormat format) {
Chris Forbes47567b72017-06-09 12:09:45 -0700488 switch (format) {
489 case VK_FORMAT_R64G64B64A64_SFLOAT:
490 case VK_FORMAT_R64G64B64A64_SINT:
491 case VK_FORMAT_R64G64B64A64_UINT:
492 case VK_FORMAT_R64G64B64_SFLOAT:
493 case VK_FORMAT_R64G64B64_SINT:
494 case VK_FORMAT_R64G64B64_UINT:
495 return 2;
496 default:
497 return 1;
498 }
499}
500
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600501static unsigned GetFormatType(VkFormat fmt) {
Dave Houltona9df0ce2018-02-07 10:51:23 -0700502 if (FormatIsSInt(fmt)) return FORMAT_TYPE_SINT;
503 if (FormatIsUInt(fmt)) return FORMAT_TYPE_UINT;
504 if (FormatIsDepthAndStencil(fmt)) return FORMAT_TYPE_FLOAT | FORMAT_TYPE_UINT;
505 if (fmt == VK_FORMAT_UNDEFINED) return 0;
Chris Forbes47567b72017-06-09 12:09:45 -0700506 // everything else -- UNORM/SNORM/FLOAT/USCALED/SSCALED is all float in the shader.
507 return FORMAT_TYPE_FLOAT;
508}
509
510// characterizes a SPIR-V type appearing in an interface to a FF stage, for comparison to a VkFormat's characterization above.
Chris Forbes062f1222018-08-21 15:34:15 -0700511// also used for input attachments, as we statically know their format.
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600512static unsigned GetFundamentalType(SHADER_MODULE_STATE const *src, unsigned type) {
Chris Forbes47567b72017-06-09 12:09:45 -0700513 auto insn = src->get_def(type);
514 assert(insn != src->end());
515
516 switch (insn.opcode()) {
517 case spv::OpTypeInt:
518 return insn.word(3) ? FORMAT_TYPE_SINT : FORMAT_TYPE_UINT;
519 case spv::OpTypeFloat:
520 return FORMAT_TYPE_FLOAT;
521 case spv::OpTypeVector:
Chris Forbes47567b72017-06-09 12:09:45 -0700522 case spv::OpTypeMatrix:
Chris Forbes47567b72017-06-09 12:09:45 -0700523 case spv::OpTypeArray:
Chris Forbes062f1222018-08-21 15:34:15 -0700524 case spv::OpTypeRuntimeArray:
525 case spv::OpTypeImage:
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600526 return GetFundamentalType(src, insn.word(2));
Chris Forbes47567b72017-06-09 12:09:45 -0700527 case spv::OpTypePointer:
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600528 return GetFundamentalType(src, insn.word(3));
Chris Forbes47567b72017-06-09 12:09:45 -0700529
530 default:
531 return 0;
532 }
533}
534
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600535static uint32_t GetShaderStageId(VkShaderStageFlagBits stage) {
Chris Forbes47567b72017-06-09 12:09:45 -0700536 uint32_t bit_pos = uint32_t(u_ffs(stage));
537 return bit_pos - 1;
538}
539
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600540static spirv_inst_iter GetStructType(SHADER_MODULE_STATE const *src, spirv_inst_iter def, bool is_array_of_verts) {
Chris Forbes47567b72017-06-09 12:09:45 -0700541 while (true) {
542 if (def.opcode() == spv::OpTypePointer) {
543 def = src->get_def(def.word(3));
544 } else if (def.opcode() == spv::OpTypeArray && is_array_of_verts) {
545 def = src->get_def(def.word(2));
546 is_array_of_verts = false;
547 } else if (def.opcode() == spv::OpTypeStruct) {
548 return def;
549 } else {
550 return src->end();
551 }
552 }
553}
554
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600555static bool CollectInterfaceBlockMembers(SHADER_MODULE_STATE const *src, std::map<location_t, interface_var> *out,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600556 std::unordered_map<unsigned, unsigned> const &blocks, bool is_array_of_verts, uint32_t id,
557 uint32_t type_id, bool is_patch, int /*first_location*/) {
Chris Forbes47567b72017-06-09 12:09:45 -0700558 // Walk down the type_id presented, trying to determine whether it's actually an interface block.
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600559 auto type = GetStructType(src, src->get_def(type_id), is_array_of_verts && !is_patch);
Chris Forbes47567b72017-06-09 12:09:45 -0700560 if (type == src->end() || blocks.find(type.word(1)) == blocks.end()) {
561 // This isn't an interface block.
Chris Forbesa313d772017-06-13 13:59:41 -0700562 return false;
Chris Forbes47567b72017-06-09 12:09:45 -0700563 }
564
565 std::unordered_map<unsigned, unsigned> member_components;
566 std::unordered_map<unsigned, unsigned> member_relaxed_precision;
Chris Forbesa313d772017-06-13 13:59:41 -0700567 std::unordered_map<unsigned, unsigned> member_patch;
Chris Forbes47567b72017-06-09 12:09:45 -0700568
569 // Walk all the OpMemberDecorate for type's result id -- first pass, collect components.
570 for (auto insn : *src) {
571 if (insn.opcode() == spv::OpMemberDecorate && insn.word(1) == type.word(1)) {
572 unsigned member_index = insn.word(2);
573
574 if (insn.word(3) == spv::DecorationComponent) {
575 unsigned component = insn.word(4);
576 member_components[member_index] = component;
577 }
578
579 if (insn.word(3) == spv::DecorationRelaxedPrecision) {
580 member_relaxed_precision[member_index] = 1;
581 }
Chris Forbesa313d772017-06-13 13:59:41 -0700582
583 if (insn.word(3) == spv::DecorationPatch) {
584 member_patch[member_index] = 1;
585 }
Chris Forbes47567b72017-06-09 12:09:45 -0700586 }
587 }
588
Chris Forbesa313d772017-06-13 13:59:41 -0700589 // TODO: correctly handle location assignment from outside
590
Chris Forbes47567b72017-06-09 12:09:45 -0700591 // Second pass -- produce the output, from Location decorations
592 for (auto insn : *src) {
593 if (insn.opcode() == spv::OpMemberDecorate && insn.word(1) == type.word(1)) {
594 unsigned member_index = insn.word(2);
595 unsigned member_type_id = type.word(2 + member_index);
596
597 if (insn.word(3) == spv::DecorationLocation) {
598 unsigned location = insn.word(4);
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600599 unsigned num_locations = GetLocationsConsumedByType(src, member_type_id, false);
Chris Forbes47567b72017-06-09 12:09:45 -0700600 auto component_it = member_components.find(member_index);
601 unsigned component = component_it == member_components.end() ? 0 : component_it->second;
602 bool is_relaxed_precision = member_relaxed_precision.find(member_index) != member_relaxed_precision.end();
Dave Houltona9df0ce2018-02-07 10:51:23 -0700603 bool member_is_patch = is_patch || member_patch.count(member_index) > 0;
Chris Forbes47567b72017-06-09 12:09:45 -0700604
605 for (unsigned int offset = 0; offset < num_locations; offset++) {
606 interface_var v = {};
607 v.id = id;
608 // TODO: member index in interface_var too?
609 v.type_id = member_type_id;
610 v.offset = offset;
Chris Forbesa313d772017-06-13 13:59:41 -0700611 v.is_patch = member_is_patch;
Chris Forbes47567b72017-06-09 12:09:45 -0700612 v.is_block_member = true;
613 v.is_relaxed_precision = is_relaxed_precision;
614 (*out)[std::make_pair(location + offset, component)] = v;
615 }
616 }
617 }
618 }
Chris Forbesa313d772017-06-13 13:59:41 -0700619
620 return true;
Chris Forbes47567b72017-06-09 12:09:45 -0700621}
622
Ari Suonpaa696b3432019-03-11 14:02:57 +0200623static std::vector<uint32_t> FindEntrypointInterfaces(spirv_inst_iter entrypoint) {
624 std::vector<uint32_t> interfaces;
625 // Find the end of the entrypoint's name string. additional zero bytes follow the actual null terminator, to fill out the
626 // rest of the word - so we only need to look at the last byte in the word to determine which word contains the terminator.
627 uint32_t word = 3;
628 while (entrypoint.word(word) & 0xff000000u) {
629 ++word;
630 }
631 ++word;
632
633 for (; word < entrypoint.len(); word++) interfaces.push_back(entrypoint.word(word));
634
635 return interfaces;
636}
637
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600638static std::map<location_t, interface_var> CollectInterfaceByLocation(SHADER_MODULE_STATE const *src, spirv_inst_iter entrypoint,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600639 spv::StorageClass sinterface, bool is_array_of_verts) {
Chris Forbes47567b72017-06-09 12:09:45 -0700640 std::unordered_map<unsigned, unsigned> var_locations;
641 std::unordered_map<unsigned, unsigned> var_builtins;
642 std::unordered_map<unsigned, unsigned> var_components;
643 std::unordered_map<unsigned, unsigned> blocks;
644 std::unordered_map<unsigned, unsigned> var_patch;
645 std::unordered_map<unsigned, unsigned> var_relaxed_precision;
646
647 for (auto insn : *src) {
648 // We consider two interface models: SSO rendezvous-by-location, and builtins. Complain about anything that
649 // fits neither model.
650 if (insn.opcode() == spv::OpDecorate) {
651 if (insn.word(2) == spv::DecorationLocation) {
652 var_locations[insn.word(1)] = insn.word(3);
653 }
654
655 if (insn.word(2) == spv::DecorationBuiltIn) {
656 var_builtins[insn.word(1)] = insn.word(3);
657 }
658
659 if (insn.word(2) == spv::DecorationComponent) {
660 var_components[insn.word(1)] = insn.word(3);
661 }
662
663 if (insn.word(2) == spv::DecorationBlock) {
664 blocks[insn.word(1)] = 1;
665 }
666
667 if (insn.word(2) == spv::DecorationPatch) {
668 var_patch[insn.word(1)] = 1;
669 }
670
671 if (insn.word(2) == spv::DecorationRelaxedPrecision) {
672 var_relaxed_precision[insn.word(1)] = 1;
673 }
674 }
675 }
676
677 // TODO: handle grouped decorations
678 // TODO: handle index=1 dual source outputs from FS -- two vars will have the same location, and we DON'T want to clobber.
679
Chris Forbes47567b72017-06-09 12:09:45 -0700680 std::map<location_t, interface_var> out;
681
Ari Suonpaa696b3432019-03-11 14:02:57 +0200682 for (uint32_t word : FindEntrypointInterfaces(entrypoint)) {
683 auto insn = src->get_def(word);
Chris Forbes47567b72017-06-09 12:09:45 -0700684 assert(insn != src->end());
685 assert(insn.opcode() == spv::OpVariable);
686
687 if (insn.word(3) == static_cast<uint32_t>(sinterface)) {
688 unsigned id = insn.word(2);
689 unsigned type = insn.word(1);
690
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600691 int location = ValueOrDefault(var_locations, id, static_cast<unsigned>(-1));
692 int builtin = ValueOrDefault(var_builtins, id, static_cast<unsigned>(-1));
693 unsigned component = ValueOrDefault(var_components, id, 0); // Unspecified is OK, is 0
Chris Forbes47567b72017-06-09 12:09:45 -0700694 bool is_patch = var_patch.find(id) != var_patch.end();
695 bool is_relaxed_precision = var_relaxed_precision.find(id) != var_relaxed_precision.end();
696
Dave Houltona9df0ce2018-02-07 10:51:23 -0700697 if (builtin != -1)
698 continue;
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600699 else if (!CollectInterfaceBlockMembers(src, &out, blocks, is_array_of_verts, id, type, is_patch, location)) {
Chris Forbes47567b72017-06-09 12:09:45 -0700700 // A user-defined interface variable, with a location. Where a variable occupied multiple locations, emit
701 // one result for each.
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600702 unsigned num_locations = GetLocationsConsumedByType(src, type, is_array_of_verts && !is_patch);
Chris Forbes47567b72017-06-09 12:09:45 -0700703 for (unsigned int offset = 0; offset < num_locations; offset++) {
704 interface_var v = {};
705 v.id = id;
706 v.type_id = type;
707 v.offset = offset;
708 v.is_patch = is_patch;
709 v.is_relaxed_precision = is_relaxed_precision;
710 out[std::make_pair(location + offset, component)] = v;
711 }
Chris Forbes47567b72017-06-09 12:09:45 -0700712 }
713 }
714 }
715
716 return out;
717}
718
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600719static std::vector<uint32_t> CollectBuiltinBlockMembers(SHADER_MODULE_STATE const *src, spirv_inst_iter entrypoint,
Ari Suonpaa696b3432019-03-11 14:02:57 +0200720 uint32_t storageClass) {
721 std::vector<uint32_t> variables;
722 std::vector<uint32_t> builtinStructMembers;
723 std::vector<uint32_t> builtinDecorations;
724
725 for (auto insn : *src) {
726 switch (insn.opcode()) {
727 // Find all built-in member decorations
728 case spv::OpMemberDecorate:
729 if (insn.word(3) == spv::DecorationBuiltIn) {
730 builtinStructMembers.push_back(insn.word(1));
731 }
732 break;
733 // Find all built-in decorations
734 case spv::OpDecorate:
735 switch (insn.word(2)) {
736 case spv::DecorationBlock: {
737 uint32_t blockID = insn.word(1);
738 for (auto builtInBlockID : builtinStructMembers) {
739 // Check if one of the members of the block are built-in -> the block is built-in
740 if (blockID == builtInBlockID) {
741 builtinDecorations.push_back(blockID);
742 break;
743 }
744 }
745 break;
746 }
747 case spv::DecorationBuiltIn:
748 builtinDecorations.push_back(insn.word(1));
749 break;
750 default:
751 break;
752 }
753 break;
754 default:
755 break;
756 }
757 }
758
759 // Find all interface variables belonging to the entrypoint and matching the storage class
760 for (uint32_t id : FindEntrypointInterfaces(entrypoint)) {
761 auto def = src->get_def(id);
762 assert(def != src->end());
763 assert(def.opcode() == spv::OpVariable);
764
765 if (def.word(3) == storageClass) variables.push_back(def.word(1));
766 }
767
768 // Find all members belonging to the builtin block selected
769 std::vector<uint32_t> builtinBlockMembers;
770 for (auto &var : variables) {
771 auto def = src->get_def(src->get_def(var).word(3));
772
773 // It could be an array of IO blocks. The element type should be the struct defining the block contents
774 if (def.opcode() == spv::OpTypeArray) def = src->get_def(def.word(2));
775
776 // Now find all members belonging to the struct defining the IO block
777 if (def.opcode() == spv::OpTypeStruct) {
778 for (auto builtInID : builtinDecorations) {
779 if (builtInID == def.word(1)) {
780 for (int i = 2; i < (int)def.len(); i++)
781 builtinBlockMembers.push_back(spv::BuiltInMax); // Start with undefined builtin for each struct member.
782 // These shouldn't be left after replacing.
783 for (auto insn : *src) {
784 if (insn.opcode() == spv::OpMemberDecorate && insn.word(1) == builtInID &&
785 insn.word(3) == spv::DecorationBuiltIn) {
786 auto structIndex = insn.word(2);
787 assert(structIndex < builtinBlockMembers.size());
788 builtinBlockMembers[structIndex] = insn.word(4);
789 }
790 }
791 }
792 }
793 }
794 }
795
796 return builtinBlockMembers;
797}
798
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600799static std::vector<std::pair<uint32_t, interface_var>> CollectInterfaceByInputAttachmentIndex(
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600800 SHADER_MODULE_STATE const *src, std::unordered_set<uint32_t> const &accessible_ids) {
Chris Forbes47567b72017-06-09 12:09:45 -0700801 std::vector<std::pair<uint32_t, interface_var>> out;
802
803 for (auto insn : *src) {
804 if (insn.opcode() == spv::OpDecorate) {
805 if (insn.word(2) == spv::DecorationInputAttachmentIndex) {
806 auto attachment_index = insn.word(3);
807 auto id = insn.word(1);
808
809 if (accessible_ids.count(id)) {
810 auto def = src->get_def(id);
811 assert(def != src->end());
812
813 if (def.opcode() == spv::OpVariable && insn.word(3) == spv::StorageClassUniformConstant) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600814 auto num_locations = GetLocationsConsumedByType(src, def.word(1), false);
Chris Forbes47567b72017-06-09 12:09:45 -0700815 for (unsigned int offset = 0; offset < num_locations; offset++) {
816 interface_var v = {};
817 v.id = id;
818 v.type_id = def.word(1);
819 v.offset = offset;
820 out.emplace_back(attachment_index + offset, v);
821 }
822 }
823 }
824 }
825 }
826 }
827
828 return out;
829}
830
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600831static bool IsWritableDescriptorType(SHADER_MODULE_STATE const *module, uint32_t type_id, bool is_storage_buffer) {
Chris Forbes8af24522018-03-07 11:37:45 -0800832 auto type = module->get_def(type_id);
833
834 // Strip off any array or ptrs. Where we remove array levels, adjust the descriptor count for each dimension.
Chris Forbes062f1222018-08-21 15:34:15 -0700835 while (type.opcode() == spv::OpTypeArray || type.opcode() == spv::OpTypePointer || type.opcode() == spv::OpTypeRuntimeArray) {
836 if (type.opcode() == spv::OpTypeArray || type.opcode() == spv::OpTypeRuntimeArray) {
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700837 type = module->get_def(type.word(2)); // Element type
Chris Forbes8af24522018-03-07 11:37:45 -0800838 } else {
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700839 type = module->get_def(type.word(3)); // Pointee type
Chris Forbes8af24522018-03-07 11:37:45 -0800840 }
841 }
842
843 switch (type.opcode()) {
844 case spv::OpTypeImage: {
845 auto dim = type.word(3);
846 auto sampled = type.word(7);
847 return sampled == 2 && dim != spv::DimSubpassData;
848 }
849
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700850 case spv::OpTypeStruct: {
851 std::unordered_set<unsigned> nonwritable_members;
Chris Forbes8af24522018-03-07 11:37:45 -0800852 for (auto insn : *module) {
853 if (insn.opcode() == spv::OpDecorate && insn.word(1) == type.word(1)) {
854 if (insn.word(2) == spv::DecorationBufferBlock) {
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700855 // Legacy storage block in the Uniform storage class
856 // has its struct type decorated with BufferBlock.
857 is_storage_buffer = true;
Chris Forbes8af24522018-03-07 11:37:45 -0800858 }
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700859 } else if (insn.opcode() == spv::OpMemberDecorate && insn.word(1) == type.word(1) &&
860 insn.word(3) == spv::DecorationNonWritable) {
861 nonwritable_members.insert(insn.word(2));
Chris Forbes8af24522018-03-07 11:37:45 -0800862 }
863 }
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700864
865 // A buffer is writable if it's either flavor of storage buffer, and has any member not decorated
866 // as nonwritable.
867 return is_storage_buffer && nonwritable_members.size() != type.len() - 2;
868 }
Chris Forbes8af24522018-03-07 11:37:45 -0800869 }
870
871 return false;
872}
873
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600874static std::vector<std::pair<descriptor_slot_t, interface_var>> CollectInterfaceByDescriptorSlot(
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600875 debug_report_data const *report_data, SHADER_MODULE_STATE const *src, std::unordered_set<uint32_t> const &accessible_ids,
Chris Forbes8af24522018-03-07 11:37:45 -0800876 bool *has_writable_descriptor) {
Chris Forbes47567b72017-06-09 12:09:45 -0700877 std::unordered_map<unsigned, unsigned> var_sets;
878 std::unordered_map<unsigned, unsigned> var_bindings;
Chris Forbes8af24522018-03-07 11:37:45 -0800879 std::unordered_map<unsigned, unsigned> var_nonwritable;
Chris Forbes47567b72017-06-09 12:09:45 -0700880
881 for (auto insn : *src) {
882 // All variables in the Uniform or UniformConstant storage classes are required to be decorated with both
883 // DecorationDescriptorSet and DecorationBinding.
884 if (insn.opcode() == spv::OpDecorate) {
885 if (insn.word(2) == spv::DecorationDescriptorSet) {
886 var_sets[insn.word(1)] = insn.word(3);
887 }
888
889 if (insn.word(2) == spv::DecorationBinding) {
890 var_bindings[insn.word(1)] = insn.word(3);
891 }
Chris Forbes8af24522018-03-07 11:37:45 -0800892
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700893 // Note: do toplevel DecorationNonWritable out here; it applies to
894 // the OpVariable rather than the type.
Chris Forbes8af24522018-03-07 11:37:45 -0800895 if (insn.word(2) == spv::DecorationNonWritable) {
896 var_nonwritable[insn.word(1)] = 1;
897 }
Chris Forbes47567b72017-06-09 12:09:45 -0700898 }
899 }
900
901 std::vector<std::pair<descriptor_slot_t, interface_var>> out;
902
903 for (auto id : accessible_ids) {
904 auto insn = src->get_def(id);
905 assert(insn != src->end());
906
907 if (insn.opcode() == spv::OpVariable &&
Chris Forbes9f89d752018-03-07 12:57:48 -0800908 (insn.word(3) == spv::StorageClassUniform || insn.word(3) == spv::StorageClassUniformConstant ||
909 insn.word(3) == spv::StorageClassStorageBuffer)) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600910 unsigned set = ValueOrDefault(var_sets, insn.word(2), 0);
911 unsigned binding = ValueOrDefault(var_bindings, insn.word(2), 0);
Chris Forbes47567b72017-06-09 12:09:45 -0700912
913 interface_var v = {};
914 v.id = insn.word(2);
915 v.type_id = insn.word(1);
916 out.emplace_back(std::make_pair(set, binding), v);
Chris Forbes8af24522018-03-07 11:37:45 -0800917
Chris Forbes8d31e5d2018-10-08 17:19:15 -0700918 if (var_nonwritable.find(id) == var_nonwritable.end() &&
919 IsWritableDescriptorType(src, insn.word(1), insn.word(3) == spv::StorageClassStorageBuffer)) {
Chris Forbes8af24522018-03-07 11:37:45 -0800920 *has_writable_descriptor = true;
921 }
Chris Forbes47567b72017-06-09 12:09:45 -0700922 }
923 }
924
925 return out;
926}
927
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600928static bool ValidateViConsistency(debug_report_data const *report_data, VkPipelineVertexInputStateCreateInfo const *vi) {
Chris Forbes47567b72017-06-09 12:09:45 -0700929 // Walk the binding descriptions, which describe the step rate and stride of each vertex buffer. Each binding should
930 // be specified only once.
931 std::unordered_map<uint32_t, VkVertexInputBindingDescription const *> bindings;
932 bool skip = false;
933
934 for (unsigned i = 0; i < vi->vertexBindingDescriptionCount; i++) {
935 auto desc = &vi->pVertexBindingDescriptions[i];
936 auto &binding = bindings[desc->binding];
937 if (binding) {
Dave Houlton78d09922018-05-17 15:48:45 -0600938 // TODO: "VUID-VkGraphicsPipelineCreateInfo-pStages-00742" perhaps?
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -0600939 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -0600940 kVUID_Core_Shader_InconsistentVi, "Duplicate vertex input binding descriptions for binding %d",
Chris Forbes47567b72017-06-09 12:09:45 -0700941 desc->binding);
942 } else {
943 binding = desc;
944 }
945 }
946
947 return skip;
948}
949
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600950static bool ValidateViAgainstVsInputs(debug_report_data const *report_data, VkPipelineVertexInputStateCreateInfo const *vi,
Mark Lobodzinski3c59d972019-04-25 11:28:14 -0600951 SHADER_MODULE_STATE const *vs, spirv_inst_iter entrypoint) {
Chris Forbes47567b72017-06-09 12:09:45 -0700952 bool skip = false;
953
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600954 auto inputs = CollectInterfaceByLocation(vs, entrypoint, spv::StorageClassInput, false);
Chris Forbes47567b72017-06-09 12:09:45 -0700955
956 // Build index by location
957 std::map<uint32_t, VkVertexInputAttributeDescription const *> attribs;
958 if (vi) {
959 for (unsigned i = 0; i < vi->vertexAttributeDescriptionCount; i++) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600960 auto num_locations = GetLocationsConsumedByFormat(vi->pVertexAttributeDescriptions[i].format);
Chris Forbes47567b72017-06-09 12:09:45 -0700961 for (auto j = 0u; j < num_locations; j++) {
962 attribs[vi->pVertexAttributeDescriptions[i].location + j] = &vi->pVertexAttributeDescriptions[i];
963 }
964 }
965 }
966
967 auto it_a = attribs.begin();
968 auto it_b = inputs.begin();
969 bool used = false;
970
971 while ((attribs.size() > 0 && it_a != attribs.end()) || (inputs.size() > 0 && it_b != inputs.end())) {
972 bool a_at_end = attribs.size() == 0 || it_a == attribs.end();
973 bool b_at_end = inputs.size() == 0 || it_b == inputs.end();
974 auto a_first = a_at_end ? 0 : it_a->first;
975 auto b_first = b_at_end ? 0 : it_b->first.first;
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -0600976
Chris Forbes47567b72017-06-09 12:09:45 -0700977 if (!a_at_end && (b_at_end || a_first < b_first)) {
Mark Young4e919b22018-05-21 15:53:59 -0600978 if (!used &&
979 log_msg(report_data, VK_DEBUG_REPORT_PERFORMANCE_WARNING_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -0600980 HandleToUint64(vs->vk_shader_module), kVUID_Core_Shader_OutputNotConsumed,
Mark Young4e919b22018-05-21 15:53:59 -0600981 "Vertex attribute at location %d not consumed by vertex shader", a_first)) {
Chris Forbes47567b72017-06-09 12:09:45 -0700982 skip = true;
983 }
984 used = false;
985 it_a++;
986 } else if (!b_at_end && (a_at_end || b_first < a_first)) {
Mark Young4e919b22018-05-21 15:53:59 -0600987 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -0600988 HandleToUint64(vs->vk_shader_module), kVUID_Core_Shader_InputNotProduced,
Mark Young4e919b22018-05-21 15:53:59 -0600989 "Vertex shader consumes input at location %d but not provided", b_first);
Chris Forbes47567b72017-06-09 12:09:45 -0700990 it_b++;
991 } else {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -0600992 unsigned attrib_type = GetFormatType(it_a->second->format);
993 unsigned input_type = GetFundamentalType(vs, it_b->second.type_id);
Chris Forbes47567b72017-06-09 12:09:45 -0700994
995 // Type checking
996 if (!(attrib_type & input_type)) {
Mark Young4e919b22018-05-21 15:53:59 -0600997 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -0600998 HandleToUint64(vs->vk_shader_module), kVUID_Core_Shader_InterfaceTypeMismatch,
Chris Forbes47567b72017-06-09 12:09:45 -0700999 "Attribute type of `%s` at location %d does not match vertex shader input type of `%s`",
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001000 string_VkFormat(it_a->second->format), a_first, DescribeType(vs, it_b->second.type_id).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -07001001 }
1002
1003 // OK!
1004 used = true;
1005 it_b++;
1006 }
1007 }
1008
1009 return skip;
1010}
1011
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001012static bool ValidateFsOutputsAgainstRenderPass(debug_report_data const *report_data, SHADER_MODULE_STATE const *fs,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001013 spirv_inst_iter entrypoint, PIPELINE_STATE const *pipeline, uint32_t subpass_index) {
Petr Krause91f7a12017-12-14 20:57:36 +01001014 auto rpci = pipeline->rp_state->createInfo.ptr();
Chris Forbes8bca1652017-07-20 11:10:09 -07001015
Chris Forbes47567b72017-06-09 12:09:45 -07001016 std::map<uint32_t, VkFormat> color_attachments;
1017 auto subpass = rpci->pSubpasses[subpass_index];
1018 for (auto i = 0u; i < subpass.colorAttachmentCount; ++i) {
1019 uint32_t attachment = subpass.pColorAttachments[i].attachment;
1020 if (attachment == VK_ATTACHMENT_UNUSED) continue;
1021 if (rpci->pAttachments[attachment].format != VK_FORMAT_UNDEFINED) {
1022 color_attachments[i] = rpci->pAttachments[attachment].format;
1023 }
1024 }
1025
1026 bool skip = false;
1027
1028 // TODO: dual source blend index (spv::DecIndex, zero if not provided)
1029
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001030 auto outputs = CollectInterfaceByLocation(fs, entrypoint, spv::StorageClassOutput, false);
Chris Forbes47567b72017-06-09 12:09:45 -07001031
1032 auto it_a = outputs.begin();
1033 auto it_b = color_attachments.begin();
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -06001034 bool used = false;
Ari Suonpaa412b23b2019-02-26 07:56:58 +02001035 bool alphaToCoverageEnabled = pipeline->graphicsPipelineCI.pMultisampleState != NULL &&
1036 pipeline->graphicsPipelineCI.pMultisampleState->alphaToCoverageEnable == VK_TRUE;
1037 bool locationZeroHasAlpha = false;
Chris Forbes47567b72017-06-09 12:09:45 -07001038
1039 // Walk attachment list and outputs together
1040
1041 while ((outputs.size() > 0 && it_a != outputs.end()) || (color_attachments.size() > 0 && it_b != color_attachments.end())) {
1042 bool a_at_end = outputs.size() == 0 || it_a == outputs.end();
1043 bool b_at_end = color_attachments.size() == 0 || it_b == color_attachments.end();
1044
Ari Suonpaa412b23b2019-02-26 07:56:58 +02001045 if (!a_at_end && it_a->first.first == 0 && fs->get_def(it_a->second.type_id) != fs->end() &&
1046 GetComponentsConsumedByType(fs, it_a->second.type_id, false) == 4)
1047 locationZeroHasAlpha = true;
1048
Chris Forbes47567b72017-06-09 12:09:45 -07001049 if (!a_at_end && (b_at_end || it_a->first.first < it_b->first)) {
Ari Suonpaa412b23b2019-02-26 07:56:58 +02001050 if (!alphaToCoverageEnabled || it_a->first.first != 0) {
1051 skip |= log_msg(report_data, VK_DEBUG_REPORT_WARNING_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
1052 HandleToUint64(fs->vk_shader_module), kVUID_Core_Shader_OutputNotConsumed,
1053 "fragment shader writes to output location %d with no matching attachment", it_a->first.first);
1054 }
Chris Forbes47567b72017-06-09 12:09:45 -07001055 it_a++;
1056 } else if (!b_at_end && (a_at_end || it_a->first.first > it_b->first)) {
Chris Forbesefdd4082017-07-20 11:19:16 -07001057 // Only complain if there are unmasked channels for this attachment. If the writemask is 0, it's acceptable for the
1058 // shader to not produce a matching output.
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -06001059 if (!used) {
1060 if (pipeline->attachments[it_b->first].colorWriteMask != 0) {
Chris Forbescfe4dca2018-10-05 10:15:00 -07001061 skip |= log_msg(report_data, VK_DEBUG_REPORT_WARNING_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -06001062 HandleToUint64(fs->vk_shader_module), kVUID_Core_Shader_InputNotProduced,
Chris Forbescfe4dca2018-10-05 10:15:00 -07001063 "Attachment %d not written by fragment shader; undefined values will be written to attachment",
1064 it_b->first);
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -06001065 }
Chris Forbesefdd4082017-07-20 11:19:16 -07001066 }
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -06001067 used = false;
Chris Forbes47567b72017-06-09 12:09:45 -07001068 it_b++;
1069 } else {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001070 unsigned output_type = GetFundamentalType(fs, it_a->second.type_id);
1071 unsigned att_type = GetFormatType(it_b->second);
Chris Forbes47567b72017-06-09 12:09:45 -07001072
1073 // Type checking
1074 if (!(output_type & att_type)) {
Chris Forbescfe4dca2018-10-05 10:15:00 -07001075 skip |= log_msg(
1076 report_data, VK_DEBUG_REPORT_WARNING_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
1077 HandleToUint64(fs->vk_shader_module), kVUID_Core_Shader_InterfaceTypeMismatch,
1078 "Attachment %d of type `%s` does not match fragment shader output type of `%s`; resulting values are undefined",
1079 it_b->first, string_VkFormat(it_b->second), DescribeType(fs, it_a->second.type_id).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -07001080 }
1081
1082 // OK!
1083 it_a++;
Mark Lobodzinski7caa39c2018-07-25 15:48:34 -06001084 used = true;
Chris Forbes47567b72017-06-09 12:09:45 -07001085 }
1086 }
1087
Ari Suonpaa412b23b2019-02-26 07:56:58 +02001088 if (alphaToCoverageEnabled && !locationZeroHasAlpha) {
1089 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
1090 HandleToUint64(fs->vk_shader_module), kVUID_Core_Shader_NoAlphaAtLocation0WithAlphaToCoverage,
1091 "fragment shader doesn't declare alpha output at location 0 even though alpha to coverage is enabled.");
1092 }
1093
Chris Forbes47567b72017-06-09 12:09:45 -07001094 return skip;
1095}
1096
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06001097// For PointSize analysis we need to know if the variable decorated with the PointSize built-in was actually written to.
1098// This function examines instructions in the static call tree for a write to this variable.
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001099static bool IsPointSizeWritten(SHADER_MODULE_STATE const *src, spirv_inst_iter builtin_instr, spirv_inst_iter entrypoint) {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06001100 auto type = builtin_instr.opcode();
1101 uint32_t target_id = builtin_instr.word(1);
1102 bool init_complete = false;
1103
1104 if (type == spv::OpMemberDecorate) {
1105 // Built-in is part of a structure -- examine instructions up to first function body to get initial IDs
1106 auto insn = entrypoint;
1107 while (!init_complete && (insn.opcode() != spv::OpFunction)) {
1108 switch (insn.opcode()) {
1109 case spv::OpTypePointer:
1110 if ((insn.word(3) == target_id) && (insn.word(2) == spv::StorageClassOutput)) {
1111 target_id = insn.word(1);
1112 }
1113 break;
1114 case spv::OpVariable:
1115 if (insn.word(1) == target_id) {
1116 target_id = insn.word(2);
1117 init_complete = true;
1118 }
1119 break;
1120 }
1121 insn++;
1122 }
1123 }
1124
Mark Lobodzinskif84b0b42018-09-11 14:54:32 -06001125 if (!init_complete && (type == spv::OpMemberDecorate)) return false;
1126
1127 bool found_write = false;
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06001128 std::unordered_set<uint32_t> worklist;
1129 worklist.insert(entrypoint.word(2));
1130
1131 // Follow instructions in call graph looking for writes to target
1132 while (!worklist.empty() && !found_write) {
1133 auto id_iter = worklist.begin();
1134 auto id = *id_iter;
1135 worklist.erase(id_iter);
1136
1137 auto insn = src->get_def(id);
1138 if (insn == src->end()) {
1139 continue;
1140 }
1141
1142 if (insn.opcode() == spv::OpFunction) {
1143 // Scan body of function looking for other function calls or items in our ID chain
1144 while (++insn, insn.opcode() != spv::OpFunctionEnd) {
1145 switch (insn.opcode()) {
1146 case spv::OpAccessChain:
1147 if (insn.word(3) == target_id) {
1148 if (type == spv::OpMemberDecorate) {
1149 auto value = GetConstantValue(src, insn.word(4));
1150 if (value == builtin_instr.word(2)) {
1151 target_id = insn.word(2);
1152 }
1153 } else {
1154 target_id = insn.word(2);
1155 }
1156 }
1157 break;
1158 case spv::OpStore:
1159 if (insn.word(1) == target_id) {
1160 found_write = true;
1161 }
1162 break;
1163 case spv::OpFunctionCall:
1164 worklist.insert(insn.word(3));
1165 break;
1166 }
1167 }
1168 }
1169 }
1170 return found_write;
1171}
1172
Chris Forbes47567b72017-06-09 12:09:45 -07001173// For some analyses, we need to know about all ids referenced by the static call tree of a particular entrypoint. This is
1174// important for identifying the set of shader resources actually used by an entrypoint, for example.
1175// Note: we only explore parts of the image which might actually contain ids we care about for the above analyses.
1176// - NOT the shader input/output interfaces.
1177//
1178// TODO: The set of interesting opcodes here was determined by eyeballing the SPIRV spec. It might be worth
1179// converting parts of this to be generated from the machine-readable spec instead.
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001180static std::unordered_set<uint32_t> MarkAccessibleIds(SHADER_MODULE_STATE const *src, spirv_inst_iter entrypoint) {
Chris Forbes47567b72017-06-09 12:09:45 -07001181 std::unordered_set<uint32_t> ids;
1182 std::unordered_set<uint32_t> worklist;
1183 worklist.insert(entrypoint.word(2));
1184
1185 while (!worklist.empty()) {
1186 auto id_iter = worklist.begin();
1187 auto id = *id_iter;
1188 worklist.erase(id_iter);
1189
1190 auto insn = src->get_def(id);
1191 if (insn == src->end()) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001192 // ID is something we didn't collect in BuildDefIndex. that's OK -- we'll stumble across all kinds of things here
Chris Forbes47567b72017-06-09 12:09:45 -07001193 // that we may not care about.
1194 continue;
1195 }
1196
1197 // Try to add to the output set
1198 if (!ids.insert(id).second) {
1199 continue; // If we already saw this id, we don't want to walk it again.
1200 }
1201
1202 switch (insn.opcode()) {
1203 case spv::OpFunction:
1204 // Scan whole body of the function, enlisting anything interesting
1205 while (++insn, insn.opcode() != spv::OpFunctionEnd) {
1206 switch (insn.opcode()) {
1207 case spv::OpLoad:
1208 case spv::OpAtomicLoad:
1209 case spv::OpAtomicExchange:
1210 case spv::OpAtomicCompareExchange:
1211 case spv::OpAtomicCompareExchangeWeak:
1212 case spv::OpAtomicIIncrement:
1213 case spv::OpAtomicIDecrement:
1214 case spv::OpAtomicIAdd:
1215 case spv::OpAtomicISub:
1216 case spv::OpAtomicSMin:
1217 case spv::OpAtomicUMin:
1218 case spv::OpAtomicSMax:
1219 case spv::OpAtomicUMax:
1220 case spv::OpAtomicAnd:
1221 case spv::OpAtomicOr:
1222 case spv::OpAtomicXor:
1223 worklist.insert(insn.word(3)); // ptr
1224 break;
1225 case spv::OpStore:
1226 case spv::OpAtomicStore:
1227 worklist.insert(insn.word(1)); // ptr
1228 break;
1229 case spv::OpAccessChain:
1230 case spv::OpInBoundsAccessChain:
1231 worklist.insert(insn.word(3)); // base ptr
1232 break;
1233 case spv::OpSampledImage:
1234 case spv::OpImageSampleImplicitLod:
1235 case spv::OpImageSampleExplicitLod:
1236 case spv::OpImageSampleDrefImplicitLod:
1237 case spv::OpImageSampleDrefExplicitLod:
1238 case spv::OpImageSampleProjImplicitLod:
1239 case spv::OpImageSampleProjExplicitLod:
1240 case spv::OpImageSampleProjDrefImplicitLod:
1241 case spv::OpImageSampleProjDrefExplicitLod:
1242 case spv::OpImageFetch:
1243 case spv::OpImageGather:
1244 case spv::OpImageDrefGather:
1245 case spv::OpImageRead:
1246 case spv::OpImage:
1247 case spv::OpImageQueryFormat:
1248 case spv::OpImageQueryOrder:
1249 case spv::OpImageQuerySizeLod:
1250 case spv::OpImageQuerySize:
1251 case spv::OpImageQueryLod:
1252 case spv::OpImageQueryLevels:
1253 case spv::OpImageQuerySamples:
1254 case spv::OpImageSparseSampleImplicitLod:
1255 case spv::OpImageSparseSampleExplicitLod:
1256 case spv::OpImageSparseSampleDrefImplicitLod:
1257 case spv::OpImageSparseSampleDrefExplicitLod:
1258 case spv::OpImageSparseSampleProjImplicitLod:
1259 case spv::OpImageSparseSampleProjExplicitLod:
1260 case spv::OpImageSparseSampleProjDrefImplicitLod:
1261 case spv::OpImageSparseSampleProjDrefExplicitLod:
1262 case spv::OpImageSparseFetch:
1263 case spv::OpImageSparseGather:
1264 case spv::OpImageSparseDrefGather:
1265 case spv::OpImageTexelPointer:
1266 worklist.insert(insn.word(3)); // Image or sampled image
1267 break;
1268 case spv::OpImageWrite:
1269 worklist.insert(insn.word(1)); // Image -- different operand order to above
1270 break;
1271 case spv::OpFunctionCall:
1272 for (uint32_t i = 3; i < insn.len(); i++) {
1273 worklist.insert(insn.word(i)); // fn itself, and all args
1274 }
1275 break;
1276
1277 case spv::OpExtInst:
1278 for (uint32_t i = 5; i < insn.len(); i++) {
1279 worklist.insert(insn.word(i)); // Operands to ext inst
1280 }
1281 break;
1282 }
1283 }
1284 break;
1285 }
1286 }
1287
1288 return ids;
1289}
1290
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001291static bool ValidatePushConstantBlockAgainstPipeline(debug_report_data const *report_data,
1292 std::vector<VkPushConstantRange> const *push_constant_ranges,
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001293 SHADER_MODULE_STATE const *src, spirv_inst_iter type,
1294 VkShaderStageFlagBits stage) {
Chris Forbes47567b72017-06-09 12:09:45 -07001295 bool skip = false;
1296
1297 // Strip off ptrs etc
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001298 type = GetStructType(src, type, false);
Chris Forbes47567b72017-06-09 12:09:45 -07001299 assert(type != src->end());
1300
1301 // Validate directly off the offsets. this isn't quite correct for arrays and matrices, but is a good first step.
1302 // TODO: arrays, matrices, weird sizes
1303 for (auto insn : *src) {
1304 if (insn.opcode() == spv::OpMemberDecorate && insn.word(1) == type.word(1)) {
1305 if (insn.word(3) == spv::DecorationOffset) {
1306 unsigned offset = insn.word(4);
1307 auto size = 4; // Bytes; TODO: calculate this based on the type
1308
1309 bool found_range = false;
1310 for (auto const &range : *push_constant_ranges) {
1311 if (range.offset <= offset && range.offset + range.size >= offset + size) {
1312 found_range = true;
1313
1314 if ((range.stageFlags & stage) == 0) {
Dave Houltona9df0ce2018-02-07 10:51:23 -07001315 skip |=
1316 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06001317 kVUID_Core_Shader_PushConstantNotAccessibleFromStage,
Dave Houltona9df0ce2018-02-07 10:51:23 -07001318 "Push constant range covering variable starting at offset %u not accessible from stage %s",
1319 offset, string_VkShaderStageFlagBits(stage));
Chris Forbes47567b72017-06-09 12:09:45 -07001320 }
1321
1322 break;
1323 }
1324 }
1325
1326 if (!found_range) {
1327 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06001328 kVUID_Core_Shader_PushConstantOutOfRange,
Dave Houltona9df0ce2018-02-07 10:51:23 -07001329 "Push constant range covering variable starting at offset %u not declared in layout", offset);
Chris Forbes47567b72017-06-09 12:09:45 -07001330 }
1331 }
1332 }
1333 }
1334
1335 return skip;
1336}
1337
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001338static bool ValidatePushConstantUsage(debug_report_data const *report_data,
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001339 std::vector<VkPushConstantRange> const *push_constant_ranges, SHADER_MODULE_STATE const *src,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001340 std::unordered_set<uint32_t> accessible_ids, VkShaderStageFlagBits stage) {
Chris Forbes47567b72017-06-09 12:09:45 -07001341 bool skip = false;
1342
1343 for (auto id : accessible_ids) {
1344 auto def_insn = src->get_def(id);
1345 if (def_insn.opcode() == spv::OpVariable && def_insn.word(3) == spv::StorageClassPushConstant) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001346 skip |= ValidatePushConstantBlockAgainstPipeline(report_data, push_constant_ranges, src, src->get_def(def_insn.word(1)),
1347 stage);
Chris Forbes47567b72017-06-09 12:09:45 -07001348 }
1349 }
1350
1351 return skip;
1352}
1353
1354// Validate that data for each specialization entry is fully contained within the buffer.
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001355static bool ValidateSpecializationOffsets(debug_report_data const *report_data, VkPipelineShaderStageCreateInfo const *info) {
Chris Forbes47567b72017-06-09 12:09:45 -07001356 bool skip = false;
1357
1358 VkSpecializationInfo const *spec = info->pSpecializationInfo;
1359
1360 if (spec) {
1361 for (auto i = 0u; i < spec->mapEntryCount; i++) {
Dave Houlton78d09922018-05-17 15:48:45 -06001362 // TODO: This is a good place for "VUID-VkSpecializationInfo-offset-00773".
Chris Forbes47567b72017-06-09 12:09:45 -07001363 if (spec->pMapEntries[i].offset + spec->pMapEntries[i].size > spec->dataSize) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06001364 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_DEVICE_EXT, 0,
Dave Houlton78d09922018-05-17 15:48:45 -06001365 "VUID-VkSpecializationInfo-pMapEntries-00774",
Dave Houltona9df0ce2018-02-07 10:51:23 -07001366 "Specialization entry %u (for constant id %u) references memory outside provided specialization "
Mark Lobodzinski487a0d12018-03-30 10:09:03 -06001367 "data (bytes %u.." PRINTF_SIZE_T_SPECIFIER "; " PRINTF_SIZE_T_SPECIFIER " bytes provided)..",
Dave Houltona9df0ce2018-02-07 10:51:23 -07001368 i, spec->pMapEntries[i].constantID, spec->pMapEntries[i].offset,
Mark Lobodzinski487a0d12018-03-30 10:09:03 -06001369 spec->pMapEntries[i].offset + spec->pMapEntries[i].size - 1, spec->dataSize);
Chris Forbes47567b72017-06-09 12:09:45 -07001370 }
1371 }
1372 }
1373
1374 return skip;
1375}
1376
Jeff Bolz38b3ce72018-09-19 12:53:38 -05001377// TODO (jbolz): Can this return a const reference?
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001378static std::set<uint32_t> TypeToDescriptorTypeSet(SHADER_MODULE_STATE const *module, uint32_t type_id, unsigned &descriptor_count) {
Chris Forbes47567b72017-06-09 12:09:45 -07001379 auto type = module->get_def(type_id);
Chris Forbes9f89d752018-03-07 12:57:48 -08001380 bool is_storage_buffer = false;
Chris Forbes47567b72017-06-09 12:09:45 -07001381 descriptor_count = 1;
Jeff Bolze54ae892018-09-08 12:16:29 -05001382 std::set<uint32_t> ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001383
1384 // Strip off any array or ptrs. Where we remove array levels, adjust the descriptor count for each dimension.
Jeff Bolzfdf96072018-04-10 14:32:18 -05001385 while (type.opcode() == spv::OpTypeArray || type.opcode() == spv::OpTypePointer || type.opcode() == spv::OpTypeRuntimeArray) {
1386 if (type.opcode() == spv::OpTypeRuntimeArray) {
1387 descriptor_count = 0;
1388 type = module->get_def(type.word(2));
1389 } else if (type.opcode() == spv::OpTypeArray) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001390 descriptor_count *= GetConstantValue(module, type.word(3));
Chris Forbes47567b72017-06-09 12:09:45 -07001391 type = module->get_def(type.word(2));
1392 } else {
Chris Forbes9f89d752018-03-07 12:57:48 -08001393 if (type.word(2) == spv::StorageClassStorageBuffer) {
1394 is_storage_buffer = true;
1395 }
Chris Forbes47567b72017-06-09 12:09:45 -07001396 type = module->get_def(type.word(3));
1397 }
1398 }
1399
1400 switch (type.opcode()) {
1401 case spv::OpTypeStruct: {
1402 for (auto insn : *module) {
1403 if (insn.opcode() == spv::OpDecorate && insn.word(1) == type.word(1)) {
1404 if (insn.word(2) == spv::DecorationBlock) {
Chris Forbes9f89d752018-03-07 12:57:48 -08001405 if (is_storage_buffer) {
Jeff Bolze54ae892018-09-08 12:16:29 -05001406 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER);
1407 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC);
1408 return ret;
Chris Forbes9f89d752018-03-07 12:57:48 -08001409 } else {
Jeff Bolze54ae892018-09-08 12:16:29 -05001410 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER);
1411 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC);
1412 ret.insert(VK_DESCRIPTOR_TYPE_INLINE_UNIFORM_BLOCK_EXT);
1413 return ret;
Chris Forbes9f89d752018-03-07 12:57:48 -08001414 }
Chris Forbes47567b72017-06-09 12:09:45 -07001415 } else if (insn.word(2) == spv::DecorationBufferBlock) {
Jeff Bolze54ae892018-09-08 12:16:29 -05001416 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER);
1417 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC);
1418 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001419 }
1420 }
1421 }
1422
1423 // Invalid
Jeff Bolze54ae892018-09-08 12:16:29 -05001424 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001425 }
1426
1427 case spv::OpTypeSampler:
Jeff Bolze54ae892018-09-08 12:16:29 -05001428 ret.insert(VK_DESCRIPTOR_TYPE_SAMPLER);
1429 ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER);
1430 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001431
Chris Forbes73c00bf2018-06-22 16:28:06 -07001432 case spv::OpTypeSampledImage: {
1433 // Slight relaxation for some GLSL historical madness: samplerBuffer doesn't really have a sampler, and a texel
1434 // buffer descriptor doesn't really provide one. Allow this slight mismatch.
1435 auto image_type = module->get_def(type.word(2));
1436 auto dim = image_type.word(3);
1437 auto sampled = image_type.word(7);
1438 if (dim == spv::DimBuffer && sampled == 1) {
Jeff Bolze54ae892018-09-08 12:16:29 -05001439 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER);
1440 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001441 }
Chris Forbes73c00bf2018-06-22 16:28:06 -07001442 }
Jeff Bolze54ae892018-09-08 12:16:29 -05001443 ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER);
1444 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001445
1446 case spv::OpTypeImage: {
1447 // Many descriptor types backing image types-- depends on dimension and whether the image will be used with a sampler.
1448 // SPIRV for Vulkan requires that sampled be 1 or 2 -- leaving the decision to runtime is unacceptable.
1449 auto dim = type.word(3);
1450 auto sampled = type.word(7);
1451
1452 if (dim == spv::DimSubpassData) {
Jeff Bolze54ae892018-09-08 12:16:29 -05001453 ret.insert(VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
1454 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001455 } else if (dim == spv::DimBuffer) {
1456 if (sampled == 1) {
Jeff Bolze54ae892018-09-08 12:16:29 -05001457 ret.insert(VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER);
1458 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001459 } else {
Jeff Bolze54ae892018-09-08 12:16:29 -05001460 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER);
1461 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001462 }
1463 } else if (sampled == 1) {
Jeff Bolze54ae892018-09-08 12:16:29 -05001464 ret.insert(VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE);
1465 ret.insert(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER);
1466 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001467 } else {
Jeff Bolze54ae892018-09-08 12:16:29 -05001468 ret.insert(VK_DESCRIPTOR_TYPE_STORAGE_IMAGE);
1469 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001470 }
1471 }
Shannon McPherson0fa28232018-11-01 11:59:02 -06001472 case spv::OpTypeAccelerationStructureNV:
Eric Werness30127fd2018-10-31 21:01:03 -07001473 ret.insert(VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_NV);
Jeff Bolz105d6492018-09-29 15:46:44 -05001474 return ret;
Chris Forbes47567b72017-06-09 12:09:45 -07001475
1476 // We shouldn't really see any other junk types -- but if we do, they're a mismatch.
1477 default:
Jeff Bolze54ae892018-09-08 12:16:29 -05001478 return ret; // Matches nothing
Chris Forbes47567b72017-06-09 12:09:45 -07001479 }
1480}
1481
Jeff Bolze54ae892018-09-08 12:16:29 -05001482static std::string string_descriptorTypes(const std::set<uint32_t> &descriptor_types) {
Chris Forbes73c00bf2018-06-22 16:28:06 -07001483 std::stringstream ss;
Jeff Bolze54ae892018-09-08 12:16:29 -05001484 for (auto it = descriptor_types.begin(); it != descriptor_types.end(); ++it) {
1485 if (ss.tellp()) ss << ", ";
1486 ss << string_VkDescriptorType(VkDescriptorType(*it));
Chris Forbes73c00bf2018-06-22 16:28:06 -07001487 }
1488 return ss.str();
1489}
1490
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001491static bool RequireFeature(debug_report_data const *report_data, VkBool32 feature, char const *feature_name) {
Chris Forbes47567b72017-06-09 12:09:45 -07001492 if (!feature) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06001493 if (log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06001494 kVUID_Core_Shader_FeatureNotEnabled, "Shader requires %s but is not enabled on the device", feature_name)) {
Chris Forbes47567b72017-06-09 12:09:45 -07001495 return true;
1496 }
1497 }
1498
1499 return false;
1500}
1501
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001502static bool RequireExtension(debug_report_data const *report_data, bool extension, char const *extension_name) {
Chris Forbes47567b72017-06-09 12:09:45 -07001503 if (!extension) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06001504 if (log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06001505 kVUID_Core_Shader_FeatureNotEnabled, "Shader requires extension %s but is not enabled on the device",
Chris Forbes47567b72017-06-09 12:09:45 -07001506 extension_name)) {
1507 return true;
1508 }
1509 }
1510
1511 return false;
1512}
1513
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001514bool CoreChecks::ValidateShaderCapabilities(SHADER_MODULE_STATE const *src, VkShaderStageFlagBits stage,
1515 bool has_writable_descriptor) {
Chris Forbes47567b72017-06-09 12:09:45 -07001516 bool skip = false;
1517
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001518 struct FeaturePointer {
1519 // Callable object to test if this feature is enabled in the given aggregate feature struct
1520 const std::function<VkBool32(const DeviceFeatures &)> IsEnabled;
1521
1522 // Test if feature pointer is populated
1523 explicit operator bool() const { return static_cast<bool>(IsEnabled); }
1524
1525 // Default and nullptr constructor to create an empty FeaturePointer
1526 FeaturePointer() : IsEnabled(nullptr) {}
1527 FeaturePointer(std::nullptr_t ptr) : IsEnabled(nullptr) {}
1528
1529 // Constructors to populate FeaturePointer based on given pointer to member
1530 FeaturePointer(VkBool32 VkPhysicalDeviceFeatures::*ptr)
1531 : IsEnabled([=](const DeviceFeatures &features) { return features.core.*ptr; }) {}
1532 FeaturePointer(VkBool32 VkPhysicalDeviceDescriptorIndexingFeaturesEXT::*ptr)
1533 : IsEnabled([=](const DeviceFeatures &features) { return features.descriptor_indexing.*ptr; }) {}
1534 FeaturePointer(VkBool32 VkPhysicalDevice8BitStorageFeaturesKHR::*ptr)
1535 : IsEnabled([=](const DeviceFeatures &features) { return features.eight_bit_storage.*ptr; }) {}
Brett Lawsonbebfb6f2018-10-23 16:58:50 -07001536 FeaturePointer(VkBool32 VkPhysicalDeviceTransformFeedbackFeaturesEXT::*ptr)
1537 : IsEnabled([=](const DeviceFeatures &features) { return features.transform_feedback_features.*ptr; }) {}
Jose-Emilio Munoz-Lopez1109b452018-08-21 09:44:07 +01001538 FeaturePointer(VkBool32 VkPhysicalDeviceFloat16Int8FeaturesKHR::*ptr)
1539 : IsEnabled([=](const DeviceFeatures &features) { return features.float16_int8.*ptr; }) {}
Tobias Hector6a0ece72018-12-10 12:24:05 +00001540 FeaturePointer(VkBool32 VkPhysicalDeviceScalarBlockLayoutFeaturesEXT::*ptr)
1541 : IsEnabled([=](const DeviceFeatures &features) { return features.scalar_block_layout_features.*ptr; }) {}
Jeff Bolze4356752019-03-07 11:23:46 -06001542 FeaturePointer(VkBool32 VkPhysicalDeviceCooperativeMatrixFeaturesNV::*ptr)
1543 : IsEnabled([=](const DeviceFeatures &features) { return features.cooperative_matrix_features.*ptr; }) {}
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001544 FeaturePointer(VkBool32 VkPhysicalDeviceFloatControlsPropertiesKHR::*ptr)
1545 : IsEnabled([=](const DeviceFeatures &features) { return features.float_controls.*ptr; }) {}
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001546 };
1547
Chris Forbes47567b72017-06-09 12:09:45 -07001548 struct CapabilityInfo {
1549 char const *name;
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001550 FeaturePointer feature;
1551 bool DeviceExtensions::*extension;
Chris Forbes47567b72017-06-09 12:09:45 -07001552 };
1553
Chris Forbes47567b72017-06-09 12:09:45 -07001554 // clang-format off
Dave Houltoneb10ea82017-12-22 12:21:50 -07001555 static const std::unordered_multimap<uint32_t, CapabilityInfo> capabilities = {
Chris Forbes47567b72017-06-09 12:09:45 -07001556 // Capabilities always supported by a Vulkan 1.0 implementation -- no
1557 // feature bits.
1558 {spv::CapabilityMatrix, {nullptr}},
1559 {spv::CapabilityShader, {nullptr}},
1560 {spv::CapabilityInputAttachment, {nullptr}},
1561 {spv::CapabilitySampled1D, {nullptr}},
1562 {spv::CapabilityImage1D, {nullptr}},
1563 {spv::CapabilitySampledBuffer, {nullptr}},
Toni Merilehtib13a4a22019-05-21 12:58:44 +03001564 {spv::CapabilityStorageImageExtendedFormats, {nullptr}},
Chris Forbes47567b72017-06-09 12:09:45 -07001565 {spv::CapabilityImageQuery, {nullptr}},
1566 {spv::CapabilityDerivativeControl, {nullptr}},
1567
1568 // Capabilities that are optionally supported, but require a feature to
1569 // be enabled on the device
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001570 {spv::CapabilityGeometry, {"VkPhysicalDeviceFeatures::geometryShader", &VkPhysicalDeviceFeatures::geometryShader}},
1571 {spv::CapabilityTessellation, {"VkPhysicalDeviceFeatures::tessellationShader", &VkPhysicalDeviceFeatures::tessellationShader}},
1572 {spv::CapabilityFloat64, {"VkPhysicalDeviceFeatures::shaderFloat64", &VkPhysicalDeviceFeatures::shaderFloat64}},
1573 {spv::CapabilityInt64, {"VkPhysicalDeviceFeatures::shaderInt64", &VkPhysicalDeviceFeatures::shaderInt64}},
1574 {spv::CapabilityTessellationPointSize, {"VkPhysicalDeviceFeatures::shaderTessellationAndGeometryPointSize", &VkPhysicalDeviceFeatures::shaderTessellationAndGeometryPointSize}},
1575 {spv::CapabilityGeometryPointSize, {"VkPhysicalDeviceFeatures::shaderTessellationAndGeometryPointSize", &VkPhysicalDeviceFeatures::shaderTessellationAndGeometryPointSize}},
1576 {spv::CapabilityImageGatherExtended, {"VkPhysicalDeviceFeatures::shaderImageGatherExtended", &VkPhysicalDeviceFeatures::shaderImageGatherExtended}},
1577 {spv::CapabilityStorageImageMultisample, {"VkPhysicalDeviceFeatures::shaderStorageImageMultisample", &VkPhysicalDeviceFeatures::shaderStorageImageMultisample}},
1578 {spv::CapabilityUniformBufferArrayDynamicIndexing, {"VkPhysicalDeviceFeatures::shaderUniformBufferArrayDynamicIndexing", &VkPhysicalDeviceFeatures::shaderUniformBufferArrayDynamicIndexing}},
1579 {spv::CapabilitySampledImageArrayDynamicIndexing, {"VkPhysicalDeviceFeatures::shaderSampledImageArrayDynamicIndexing", &VkPhysicalDeviceFeatures::shaderSampledImageArrayDynamicIndexing}},
1580 {spv::CapabilityStorageBufferArrayDynamicIndexing, {"VkPhysicalDeviceFeatures::shaderStorageBufferArrayDynamicIndexing", &VkPhysicalDeviceFeatures::shaderStorageBufferArrayDynamicIndexing}},
1581 {spv::CapabilityStorageImageArrayDynamicIndexing, {"VkPhysicalDeviceFeatures::shaderStorageImageArrayDynamicIndexing", &VkPhysicalDeviceFeatures::shaderStorageBufferArrayDynamicIndexing}},
1582 {spv::CapabilityClipDistance, {"VkPhysicalDeviceFeatures::shaderClipDistance", &VkPhysicalDeviceFeatures::shaderClipDistance}},
1583 {spv::CapabilityCullDistance, {"VkPhysicalDeviceFeatures::shaderCullDistance", &VkPhysicalDeviceFeatures::shaderCullDistance}},
1584 {spv::CapabilityImageCubeArray, {"VkPhysicalDeviceFeatures::imageCubeArray", &VkPhysicalDeviceFeatures::imageCubeArray}},
1585 {spv::CapabilitySampleRateShading, {"VkPhysicalDeviceFeatures::sampleRateShading", &VkPhysicalDeviceFeatures::sampleRateShading}},
1586 {spv::CapabilitySparseResidency, {"VkPhysicalDeviceFeatures::shaderResourceResidency", &VkPhysicalDeviceFeatures::shaderResourceResidency}},
1587 {spv::CapabilityMinLod, {"VkPhysicalDeviceFeatures::shaderResourceMinLod", &VkPhysicalDeviceFeatures::shaderResourceMinLod}},
1588 {spv::CapabilitySampledCubeArray, {"VkPhysicalDeviceFeatures::imageCubeArray", &VkPhysicalDeviceFeatures::imageCubeArray}},
1589 {spv::CapabilityImageMSArray, {"VkPhysicalDeviceFeatures::shaderStorageImageMultisample", &VkPhysicalDeviceFeatures::shaderStorageImageMultisample}},
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001590 {spv::CapabilityInterpolationFunction, {"VkPhysicalDeviceFeatures::sampleRateShading", &VkPhysicalDeviceFeatures::sampleRateShading}},
1591 {spv::CapabilityStorageImageReadWithoutFormat, {"VkPhysicalDeviceFeatures::shaderStorageImageReadWithoutFormat", &VkPhysicalDeviceFeatures::shaderStorageImageReadWithoutFormat}},
1592 {spv::CapabilityStorageImageWriteWithoutFormat, {"VkPhysicalDeviceFeatures::shaderStorageImageWriteWithoutFormat", &VkPhysicalDeviceFeatures::shaderStorageImageWriteWithoutFormat}},
1593 {spv::CapabilityMultiViewport, {"VkPhysicalDeviceFeatures::multiViewport", &VkPhysicalDeviceFeatures::multiViewport}},
Jeff Bolzfdf96072018-04-10 14:32:18 -05001594
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001595 {spv::CapabilityShaderNonUniformEXT, {VK_EXT_DESCRIPTOR_INDEXING_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_ext_descriptor_indexing}},
1596 {spv::CapabilityRuntimeDescriptorArrayEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::runtimeDescriptorArray", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::runtimeDescriptorArray}},
1597 {spv::CapabilityInputAttachmentArrayDynamicIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderInputAttachmentArrayDynamicIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderInputAttachmentArrayDynamicIndexing}},
1598 {spv::CapabilityUniformTexelBufferArrayDynamicIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderUniformTexelBufferArrayDynamicIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderUniformTexelBufferArrayDynamicIndexing}},
1599 {spv::CapabilityStorageTexelBufferArrayDynamicIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageTexelBufferArrayDynamicIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageTexelBufferArrayDynamicIndexing}},
1600 {spv::CapabilityUniformBufferArrayNonUniformIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderUniformBufferArrayNonUniformIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderUniformBufferArrayNonUniformIndexing}},
1601 {spv::CapabilitySampledImageArrayNonUniformIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderSampledImageArrayNonUniformIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderSampledImageArrayNonUniformIndexing}},
1602 {spv::CapabilityStorageBufferArrayNonUniformIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageBufferArrayNonUniformIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageBufferArrayNonUniformIndexing}},
1603 {spv::CapabilityStorageImageArrayNonUniformIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageImageArrayNonUniformIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageImageArrayNonUniformIndexing}},
1604 {spv::CapabilityInputAttachmentArrayNonUniformIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderInputAttachmentArrayNonUniformIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderInputAttachmentArrayNonUniformIndexing}},
1605 {spv::CapabilityUniformTexelBufferArrayNonUniformIndexingEXT, {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderUniformTexelBufferArrayNonUniformIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderUniformTexelBufferArrayNonUniformIndexing}},
1606 {spv::CapabilityStorageTexelBufferArrayNonUniformIndexingEXT , {"VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageTexelBufferArrayNonUniformIndexing", &VkPhysicalDeviceDescriptorIndexingFeaturesEXT::shaderStorageTexelBufferArrayNonUniformIndexing}},
Chris Forbes47567b72017-06-09 12:09:45 -07001607
1608 // Capabilities that require an extension
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001609 {spv::CapabilityDrawParameters, {VK_KHR_SHADER_DRAW_PARAMETERS_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_khr_shader_draw_parameters}},
1610 {spv::CapabilityGeometryShaderPassthroughNV, {VK_NV_GEOMETRY_SHADER_PASSTHROUGH_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_nv_geometry_shader_passthrough}},
1611 {spv::CapabilitySampleMaskOverrideCoverageNV, {VK_NV_SAMPLE_MASK_OVERRIDE_COVERAGE_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_nv_sample_mask_override_coverage}},
1612 {spv::CapabilityShaderViewportIndexLayerEXT, {VK_EXT_SHADER_VIEWPORT_INDEX_LAYER_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_ext_shader_viewport_index_layer}},
1613 {spv::CapabilityShaderViewportIndexLayerNV, {VK_NV_VIEWPORT_ARRAY2_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_nv_viewport_array2}},
1614 {spv::CapabilityShaderViewportMaskNV, {VK_NV_VIEWPORT_ARRAY2_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_nv_viewport_array2}},
1615 {spv::CapabilitySubgroupBallotKHR, {VK_EXT_SHADER_SUBGROUP_BALLOT_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_ext_shader_subgroup_ballot }},
1616 {spv::CapabilitySubgroupVoteKHR, {VK_EXT_SHADER_SUBGROUP_VOTE_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_ext_shader_subgroup_vote }},
Jason Macnakb7d091c2019-06-10 11:13:11 -07001617 {spv::CapabilityGroupNonUniformPartitionedNV, {VK_NV_SHADER_SUBGROUP_PARTITIONED_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_nv_shader_subgroup_partitioned}},
aqnuep7033c702018-09-11 18:03:29 +02001618 {spv::CapabilityInt64Atomics, {VK_KHR_SHADER_ATOMIC_INT64_EXTENSION_NAME, nullptr, &DeviceExtensions::vk_khr_shader_atomic_int64 }},
Alexander Galazin3bd8e342018-06-14 15:49:07 +02001619
Mike Schuchardt8ed5ea02018-07-20 18:24:17 -06001620 {spv::CapabilityStorageBuffer8BitAccess , {"VkPhysicalDevice8BitStorageFeaturesKHR::storageBuffer8BitAccess", &VkPhysicalDevice8BitStorageFeaturesKHR::storageBuffer8BitAccess, &DeviceExtensions::vk_khr_8bit_storage}},
1621 {spv::CapabilityUniformAndStorageBuffer8BitAccess , {"VkPhysicalDevice8BitStorageFeaturesKHR::uniformAndStorageBuffer8BitAccess", &VkPhysicalDevice8BitStorageFeaturesKHR::uniformAndStorageBuffer8BitAccess, &DeviceExtensions::vk_khr_8bit_storage}},
1622 {spv::CapabilityStoragePushConstant8 , {"VkPhysicalDevice8BitStorageFeaturesKHR::storagePushConstant8", &VkPhysicalDevice8BitStorageFeaturesKHR::storagePushConstant8, &DeviceExtensions::vk_khr_8bit_storage}},
Brett Lawsonbebfb6f2018-10-23 16:58:50 -07001623
1624 {spv::CapabilityTransformFeedback , { "VkPhysicalDeviceTransformFeedbackFeaturesEXT::transformFeedback", &VkPhysicalDeviceTransformFeedbackFeaturesEXT::transformFeedback, &DeviceExtensions::vk_ext_transform_feedback}},
Jose-Emilio Munoz-Lopez1109b452018-08-21 09:44:07 +01001625 {spv::CapabilityGeometryStreams , { "VkPhysicalDeviceTransformFeedbackFeaturesEXT::geometryStreams", &VkPhysicalDeviceTransformFeedbackFeaturesEXT::geometryStreams, &DeviceExtensions::vk_ext_transform_feedback}},
1626
1627 {spv::CapabilityFloat16 , {"VkPhysicalDeviceFloat16Int8FeaturesKHR::shaderFloat16", &VkPhysicalDeviceFloat16Int8FeaturesKHR::shaderFloat16, &DeviceExtensions::vk_khr_shader_float16_int8}},
1628 {spv::CapabilityInt8 , {"VkPhysicalDeviceFloat16Int8FeaturesKHR::shaderInt8", &VkPhysicalDeviceFloat16Int8FeaturesKHR::shaderInt8, &DeviceExtensions::vk_khr_shader_float16_int8}},
Jeff Bolze4356752019-03-07 11:23:46 -06001629
1630 {spv::CapabilityCooperativeMatrixNV, {"VkPhysicalDeviceCooperativeMatrixFeaturesNV::cooperativeMatrix", &VkPhysicalDeviceCooperativeMatrixFeaturesNV::cooperativeMatrix, &DeviceExtensions::vk_nv_cooperative_matrix}},
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00001631
1632 {spv::CapabilitySignedZeroInfNanPreserve, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderSignedZeroInfNanPreserveFloat16", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderSignedZeroInfNanPreserveFloat16, &DeviceExtensions::vk_khr_shader_float_controls}},
1633 {spv::CapabilitySignedZeroInfNanPreserve, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderSignedZeroInfNanPreserveFloat32", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderSignedZeroInfNanPreserveFloat32, &DeviceExtensions::vk_khr_shader_float_controls}},
1634 {spv::CapabilitySignedZeroInfNanPreserve, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderSignedZeroInfNanPreserveFloat64", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderSignedZeroInfNanPreserveFloat64, &DeviceExtensions::vk_khr_shader_float_controls}},
1635 {spv::CapabilityDenormPreserve, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormPreserveFloat16", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormPreserveFloat16, &DeviceExtensions::vk_khr_shader_float_controls}},
1636 {spv::CapabilityDenormPreserve, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormPreserveFloat32", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormPreserveFloat32, &DeviceExtensions::vk_khr_shader_float_controls}},
1637 {spv::CapabilityDenormPreserve, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormPreserveFloat64", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormPreserveFloat64, &DeviceExtensions::vk_khr_shader_float_controls}},
1638 {spv::CapabilityDenormFlushToZero, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormFlushToZeroFloat16", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormFlushToZeroFloat16, &DeviceExtensions::vk_khr_shader_float_controls}},
1639 {spv::CapabilityDenormFlushToZero, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormFlushToZeroFloat32", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormFlushToZeroFloat32, &DeviceExtensions::vk_khr_shader_float_controls}},
1640 {spv::CapabilityDenormFlushToZero, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormFlushToZeroFloat64", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderDenormFlushToZeroFloat64, &DeviceExtensions::vk_khr_shader_float_controls}},
1641 {spv::CapabilityRoundingModeRTE, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTEFloat16", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTEFloat16, &DeviceExtensions::vk_khr_shader_float_controls}},
1642 {spv::CapabilityRoundingModeRTE, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTEFloat32", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTEFloat32, &DeviceExtensions::vk_khr_shader_float_controls}},
1643 {spv::CapabilityRoundingModeRTE, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTEFloat64", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTEFloat64, &DeviceExtensions::vk_khr_shader_float_controls}},
1644 {spv::CapabilityRoundingModeRTZ, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTZFloat16", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTZFloat16, &DeviceExtensions::vk_khr_shader_float_controls}},
1645 {spv::CapabilityRoundingModeRTZ, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTZFloat32", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTZFloat32, &DeviceExtensions::vk_khr_shader_float_controls}},
1646 {spv::CapabilityRoundingModeRTZ, {"VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTZFloat64", &VkPhysicalDeviceFloatControlsPropertiesKHR::shaderRoundingModeRTZFloat64, &DeviceExtensions::vk_khr_shader_float_controls}},
Chris Forbes47567b72017-06-09 12:09:45 -07001647 };
1648 // clang-format on
1649
1650 for (auto insn : *src) {
1651 if (insn.opcode() == spv::OpCapability) {
Dave Houltoneb10ea82017-12-22 12:21:50 -07001652 size_t n = capabilities.count(insn.word(1));
1653 if (1 == n) { // key occurs exactly once
1654 auto it = capabilities.find(insn.word(1));
1655 if (it != capabilities.end()) {
1656 if (it->second.feature) {
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06001657 skip |= RequireFeature(report_data, it->second.feature.IsEnabled(enabled_features), it->second.name);
Dave Houltoneb10ea82017-12-22 12:21:50 -07001658 }
1659 if (it->second.extension) {
Mark Lobodzinskif45e45f2019-04-19 14:15:39 -06001660 skip |= RequireExtension(report_data, device_extensions.*(it->second.extension), it->second.name);
Dave Houltoneb10ea82017-12-22 12:21:50 -07001661 }
Chris Forbes47567b72017-06-09 12:09:45 -07001662 }
Dave Houltoneb10ea82017-12-22 12:21:50 -07001663 } else if (1 < n) { // key occurs multiple times, at least one must be enabled
1664 bool needs_feature = false, has_feature = false;
1665 bool needs_ext = false, has_ext = false;
1666 std::string feature_names = "(one of) [ ";
1667 std::string extension_names = feature_names;
1668 auto caps = capabilities.equal_range(insn.word(1));
1669 for (auto it = caps.first; it != caps.second; ++it) {
1670 if (it->second.feature) {
1671 needs_feature = true;
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06001672 has_feature = has_feature || it->second.feature.IsEnabled(enabled_features);
Dave Houltoneb10ea82017-12-22 12:21:50 -07001673 feature_names += it->second.name;
1674 feature_names += " ";
1675 }
1676 if (it->second.extension) {
1677 needs_ext = true;
Mark Lobodzinskif45e45f2019-04-19 14:15:39 -06001678 has_ext = has_ext || device_extensions.*(it->second.extension);
Dave Houltoneb10ea82017-12-22 12:21:50 -07001679 extension_names += it->second.name;
1680 extension_names += " ";
1681 }
1682 }
1683 if (needs_feature) {
1684 feature_names += "]";
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001685 skip |= RequireFeature(report_data, has_feature, feature_names.c_str());
Dave Houltoneb10ea82017-12-22 12:21:50 -07001686 }
1687 if (needs_ext) {
1688 extension_names += "]";
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06001689 skip |= RequireExtension(report_data, has_ext, extension_names.c_str());
Chris Forbes47567b72017-06-09 12:09:45 -07001690 }
1691 }
1692 }
1693 }
1694
Chris Forbes349b3132018-03-07 11:38:08 -08001695 if (has_writable_descriptor) {
1696 switch (stage) {
1697 case VK_SHADER_STAGE_COMPUTE_BIT:
Jeff Bolz148d94e2018-12-13 21:25:56 -06001698 case VK_SHADER_STAGE_RAYGEN_BIT_NV:
1699 case VK_SHADER_STAGE_ANY_HIT_BIT_NV:
1700 case VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV:
1701 case VK_SHADER_STAGE_MISS_BIT_NV:
1702 case VK_SHADER_STAGE_INTERSECTION_BIT_NV:
1703 case VK_SHADER_STAGE_CALLABLE_BIT_NV:
1704 case VK_SHADER_STAGE_TASK_BIT_NV:
1705 case VK_SHADER_STAGE_MESH_BIT_NV:
Chris Forbes349b3132018-03-07 11:38:08 -08001706 /* No feature requirements for writes and atomics from compute
Jeff Bolz148d94e2018-12-13 21:25:56 -06001707 * raytracing, or mesh stages */
Chris Forbes349b3132018-03-07 11:38:08 -08001708 break;
1709 case VK_SHADER_STAGE_FRAGMENT_BIT:
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06001710 skip |= RequireFeature(report_data, enabled_features.core.fragmentStoresAndAtomics, "fragmentStoresAndAtomics");
Chris Forbes349b3132018-03-07 11:38:08 -08001711 break;
1712 default:
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06001713 skip |= RequireFeature(report_data, enabled_features.core.vertexPipelineStoresAndAtomics,
1714 "vertexPipelineStoresAndAtomics");
Chris Forbes349b3132018-03-07 11:38:08 -08001715 break;
1716 }
1717 }
1718
Chris Forbes47567b72017-06-09 12:09:45 -07001719 return skip;
1720}
1721
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001722bool CoreChecks::ValidateShaderStageInputOutputLimits(SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001723 PIPELINE_STATE *pipeline, spirv_inst_iter entrypoint) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001724 if (pStage->stage == VK_SHADER_STAGE_COMPUTE_BIT || pStage->stage == VK_SHADER_STAGE_ALL_GRAPHICS ||
1725 pStage->stage == VK_SHADER_STAGE_ALL) {
1726 return false;
1727 }
1728
1729 bool skip = false;
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07001730 auto const &limits = phys_dev_props.limits;
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001731
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001732 std::set<uint32_t> patchIDs;
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001733 struct Variable {
1734 uint32_t baseTypePtrID;
1735 uint32_t ID;
1736 uint32_t storageClass;
1737 };
1738 std::vector<Variable> variables;
1739
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001740 uint32_t numVertices = 0;
1741
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001742 for (auto insn : *src) {
1743 switch (insn.opcode()) {
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001744 // Find all Patch decorations
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001745 case spv::OpDecorate:
1746 switch (insn.word(2)) {
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001747 case spv::DecorationPatch: {
1748 patchIDs.insert(insn.word(1));
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001749 break;
1750 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001751 default:
1752 break;
1753 }
1754 break;
1755 // Find all input and output variables
1756 case spv::OpVariable: {
1757 Variable var = {};
1758 var.storageClass = insn.word(3);
1759 if (var.storageClass == spv::StorageClassInput || var.storageClass == spv::StorageClassOutput) {
1760 var.baseTypePtrID = insn.word(1);
1761 var.ID = insn.word(2);
1762 variables.push_back(var);
1763 }
1764 break;
1765 }
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001766 case spv::OpExecutionMode:
1767 if (insn.word(1) == entrypoint.word(2)) {
1768 switch (insn.word(2)) {
1769 default:
1770 break;
1771 case spv::ExecutionModeOutputVertices:
1772 numVertices = insn.word(3);
1773 break;
1774 }
1775 }
1776 break;
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001777 default:
1778 break;
1779 }
1780 }
1781
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001782 bool strip_output_array_level =
1783 (pStage->stage == VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT || pStage->stage == VK_SHADER_STAGE_MESH_BIT_NV);
1784 bool strip_input_array_level =
1785 (pStage->stage == VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT ||
1786 pStage->stage == VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT || pStage->stage == VK_SHADER_STAGE_GEOMETRY_BIT);
1787
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001788 uint32_t numCompIn = 0, numCompOut = 0;
1789 for (auto &var : variables) {
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001790 // Check if the variable is a patch. Patches can also be members of blocks,
1791 // but if they are then the top-level arrayness has already been stripped
1792 // by the time GetComponentsConsumedByType gets to it.
1793 bool isPatch = patchIDs.find(var.ID) != patchIDs.end();
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001794
1795 if (var.storageClass == spv::StorageClassInput) {
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001796 numCompIn += GetComponentsConsumedByType(src, var.baseTypePtrID, strip_input_array_level && !isPatch);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001797 } else { // var.storageClass == spv::StorageClassOutput
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001798 numCompOut += GetComponentsConsumedByType(src, var.baseTypePtrID, strip_output_array_level && !isPatch);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001799 }
1800 }
1801
1802 switch (pStage->stage) {
1803 case VK_SHADER_STAGE_VERTEX_BIT:
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001804 if (numCompOut > limits.maxVertexOutputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001805 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1806 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1807 "Invalid Pipeline CreateInfo State: Vertex shader exceeds "
1808 "VkPhysicalDeviceLimits::maxVertexOutputComponents of %u "
1809 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001810 limits.maxVertexOutputComponents, numCompOut - limits.maxVertexOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001811 }
1812 break;
1813
1814 case VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT:
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001815 if (numCompIn > limits.maxTessellationControlPerVertexInputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001816 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1817 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1818 "Invalid Pipeline CreateInfo State: Tessellation control shader exceeds "
1819 "VkPhysicalDeviceLimits::maxTessellationControlPerVertexInputComponents of %u "
1820 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001821 limits.maxTessellationControlPerVertexInputComponents,
1822 numCompIn - limits.maxTessellationControlPerVertexInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001823 }
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001824 if (numCompOut > limits.maxTessellationControlPerVertexOutputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001825 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1826 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1827 "Invalid Pipeline CreateInfo State: Tessellation control shader exceeds "
1828 "VkPhysicalDeviceLimits::maxTessellationControlPerVertexOutputComponents of %u "
1829 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001830 limits.maxTessellationControlPerVertexOutputComponents,
1831 numCompOut - limits.maxTessellationControlPerVertexOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001832 }
1833 break;
1834
1835 case VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT:
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001836 if (numCompIn > limits.maxTessellationEvaluationInputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001837 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1838 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1839 "Invalid Pipeline CreateInfo State: Tessellation evaluation shader exceeds "
1840 "VkPhysicalDeviceLimits::maxTessellationEvaluationInputComponents of %u "
1841 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001842 limits.maxTessellationEvaluationInputComponents,
1843 numCompIn - limits.maxTessellationEvaluationInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001844 }
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001845 if (numCompOut > limits.maxTessellationEvaluationOutputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001846 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1847 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1848 "Invalid Pipeline CreateInfo State: Tessellation evaluation shader exceeds "
1849 "VkPhysicalDeviceLimits::maxTessellationEvaluationOutputComponents of %u "
1850 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001851 limits.maxTessellationEvaluationOutputComponents,
1852 numCompOut - limits.maxTessellationEvaluationOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001853 }
1854 break;
1855
1856 case VK_SHADER_STAGE_GEOMETRY_BIT:
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001857 if (numCompIn > limits.maxGeometryInputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001858 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1859 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1860 "Invalid Pipeline CreateInfo State: Geometry shader exceeds "
1861 "VkPhysicalDeviceLimits::maxGeometryInputComponents of %u "
1862 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001863 limits.maxGeometryInputComponents, numCompIn - limits.maxGeometryInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001864 }
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001865 if (numCompOut > limits.maxGeometryOutputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001866 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1867 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1868 "Invalid Pipeline CreateInfo State: Geometry shader exceeds "
1869 "VkPhysicalDeviceLimits::maxGeometryOutputComponents of %u "
1870 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001871 limits.maxGeometryOutputComponents, numCompOut - limits.maxGeometryOutputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001872 }
Jeff Bolze9ee3d82019-05-29 13:45:13 -05001873 if (numCompOut * numVertices > limits.maxGeometryTotalOutputComponents) {
1874 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1875 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1876 "Invalid Pipeline CreateInfo State: Geometry shader exceeds "
1877 "VkPhysicalDeviceLimits::maxGeometryTotalOutputComponents of %u "
1878 "components by %u components",
1879 limits.maxGeometryTotalOutputComponents,
1880 numCompOut * numVertices - limits.maxGeometryTotalOutputComponents);
1881 }
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001882 break;
1883
1884 case VK_SHADER_STAGE_FRAGMENT_BIT:
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001885 if (numCompIn > limits.maxFragmentInputComponents) {
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001886 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
1887 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_ExceedDeviceLimit,
1888 "Invalid Pipeline CreateInfo State: Fragment shader exceeds "
1889 "VkPhysicalDeviceLimits::maxFragmentInputComponents of %u "
1890 "components by %u components",
Mark Lobodzinski57a44272019-02-27 12:40:50 -07001891 limits.maxFragmentInputComponents, numCompIn - limits.maxFragmentInputComponents);
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001892 }
1893 break;
1894
Jeff Bolz148d94e2018-12-13 21:25:56 -06001895 case VK_SHADER_STAGE_RAYGEN_BIT_NV:
1896 case VK_SHADER_STAGE_ANY_HIT_BIT_NV:
1897 case VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV:
1898 case VK_SHADER_STAGE_MISS_BIT_NV:
1899 case VK_SHADER_STAGE_INTERSECTION_BIT_NV:
1900 case VK_SHADER_STAGE_CALLABLE_BIT_NV:
1901 case VK_SHADER_STAGE_TASK_BIT_NV:
1902 case VK_SHADER_STAGE_MESH_BIT_NV:
1903 break;
1904
Daniel Fedai Larsenc939abc2018-08-07 10:01:58 +02001905 default:
1906 assert(false); // This should never happen
1907 }
1908 return skip;
1909}
1910
Jeff Bolze4356752019-03-07 11:23:46 -06001911// copy the specialization constant value into buf, if it is present
1912void GetSpecConstantValue(VkPipelineShaderStageCreateInfo const *pStage, uint32_t spec_id, void *buf) {
1913 VkSpecializationInfo const *spec = pStage->pSpecializationInfo;
1914
1915 if (spec && spec_id < spec->mapEntryCount) {
1916 memcpy(buf, (uint8_t *)spec->pData + spec->pMapEntries[spec_id].offset, spec->pMapEntries[spec_id].size);
1917 }
1918}
1919
1920// Fill in value with the constant or specialization constant value, if available.
1921// Returns true if the value has been accurately filled out.
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001922static bool GetIntConstantValue(spirv_inst_iter insn, SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
Jeff Bolze4356752019-03-07 11:23:46 -06001923 const std::unordered_map<uint32_t, uint32_t> &id_to_spec_id, uint32_t *value) {
1924 auto type_id = src->get_def(insn.word(1));
1925 if (type_id.opcode() != spv::OpTypeInt || type_id.word(2) != 32) {
1926 return false;
1927 }
1928 switch (insn.opcode()) {
1929 case spv::OpSpecConstant:
1930 *value = insn.word(3);
1931 GetSpecConstantValue(pStage, id_to_spec_id.at(insn.word(2)), value);
1932 return true;
1933 case spv::OpConstant:
1934 *value = insn.word(3);
1935 return true;
1936 default:
1937 return false;
1938 }
1939}
1940
1941// Map SPIR-V type to VK_COMPONENT_TYPE enum
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001942VkComponentTypeNV GetComponentType(spirv_inst_iter insn, SHADER_MODULE_STATE const *src) {
Jeff Bolze4356752019-03-07 11:23:46 -06001943 switch (insn.opcode()) {
1944 case spv::OpTypeInt:
1945 switch (insn.word(2)) {
1946 case 8:
1947 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT8_NV : VK_COMPONENT_TYPE_UINT8_NV;
1948 case 16:
1949 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT16_NV : VK_COMPONENT_TYPE_UINT16_NV;
1950 case 32:
1951 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT32_NV : VK_COMPONENT_TYPE_UINT32_NV;
1952 case 64:
1953 return insn.word(3) != 0 ? VK_COMPONENT_TYPE_SINT64_NV : VK_COMPONENT_TYPE_UINT64_NV;
1954 default:
1955 return VK_COMPONENT_TYPE_MAX_ENUM_NV;
1956 }
1957 case spv::OpTypeFloat:
1958 switch (insn.word(2)) {
1959 case 16:
1960 return VK_COMPONENT_TYPE_FLOAT16_NV;
1961 case 32:
1962 return VK_COMPONENT_TYPE_FLOAT32_NV;
1963 case 64:
1964 return VK_COMPONENT_TYPE_FLOAT64_NV;
1965 default:
1966 return VK_COMPONENT_TYPE_MAX_ENUM_NV;
1967 }
1968 default:
1969 return VK_COMPONENT_TYPE_MAX_ENUM_NV;
1970 }
1971}
1972
1973// Validate SPV_NV_cooperative_matrix behavior that can't be statically validated
1974// in SPIRV-Tools (e.g. due to specialization constant usage).
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001975bool CoreChecks::ValidateCooperativeMatrix(SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
Jeff Bolze4356752019-03-07 11:23:46 -06001976 PIPELINE_STATE *pipeline) {
1977 bool skip = false;
1978
1979 // Map SPIR-V result ID to specialization constant id (SpecId decoration value)
1980 std::unordered_map<uint32_t, uint32_t> id_to_spec_id;
1981 // Map SPIR-V result ID to the ID of its type.
1982 std::unordered_map<uint32_t, uint32_t> id_to_type_id;
1983
1984 struct CoopMatType {
1985 uint32_t scope, rows, cols;
1986 VkComponentTypeNV component_type;
1987 bool all_constant;
1988
1989 CoopMatType() : scope(0), rows(0), cols(0), component_type(VK_COMPONENT_TYPE_MAX_ENUM_NV), all_constant(false) {}
1990
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06001991 void Init(uint32_t id, SHADER_MODULE_STATE const *src, VkPipelineShaderStageCreateInfo const *pStage,
Jeff Bolze4356752019-03-07 11:23:46 -06001992 const std::unordered_map<uint32_t, uint32_t> &id_to_spec_id) {
1993 spirv_inst_iter insn = src->get_def(id);
1994 uint32_t component_type_id = insn.word(2);
1995 uint32_t scope_id = insn.word(3);
1996 uint32_t rows_id = insn.word(4);
1997 uint32_t cols_id = insn.word(5);
1998 auto component_type_iter = src->get_def(component_type_id);
1999 auto scope_iter = src->get_def(scope_id);
2000 auto rows_iter = src->get_def(rows_id);
2001 auto cols_iter = src->get_def(cols_id);
2002
2003 all_constant = true;
2004 if (!GetIntConstantValue(scope_iter, src, pStage, id_to_spec_id, &scope)) {
2005 all_constant = false;
2006 }
2007 if (!GetIntConstantValue(rows_iter, src, pStage, id_to_spec_id, &rows)) {
2008 all_constant = false;
2009 }
2010 if (!GetIntConstantValue(cols_iter, src, pStage, id_to_spec_id, &cols)) {
2011 all_constant = false;
2012 }
2013 component_type = GetComponentType(component_type_iter, src);
2014 }
2015 };
2016
2017 bool seen_coopmat_capability = false;
2018
2019 for (auto insn : *src) {
2020 // Whitelist instructions whose result can be a cooperative matrix type, and
2021 // keep track of their types. It would be nice if SPIRV-Headers generated code
2022 // to identify which instructions have a result type and result id. Lacking that,
2023 // this whitelist is based on the set of instructions that
2024 // SPV_NV_cooperative_matrix says can be used with cooperative matrix types.
2025 switch (insn.opcode()) {
2026 case spv::OpLoad:
2027 case spv::OpCooperativeMatrixLoadNV:
2028 case spv::OpCooperativeMatrixMulAddNV:
2029 case spv::OpSNegate:
2030 case spv::OpFNegate:
2031 case spv::OpIAdd:
2032 case spv::OpFAdd:
2033 case spv::OpISub:
2034 case spv::OpFSub:
2035 case spv::OpFDiv:
2036 case spv::OpSDiv:
2037 case spv::OpUDiv:
2038 case spv::OpMatrixTimesScalar:
2039 case spv::OpConstantComposite:
2040 case spv::OpCompositeConstruct:
2041 case spv::OpConvertFToU:
2042 case spv::OpConvertFToS:
2043 case spv::OpConvertSToF:
2044 case spv::OpConvertUToF:
2045 case spv::OpUConvert:
2046 case spv::OpSConvert:
2047 case spv::OpFConvert:
2048 id_to_type_id[insn.word(2)] = insn.word(1);
2049 break;
2050 default:
2051 break;
2052 }
2053
2054 switch (insn.opcode()) {
2055 case spv::OpDecorate:
2056 if (insn.word(2) == spv::DecorationSpecId) {
2057 id_to_spec_id[insn.word(1)] = insn.word(3);
2058 }
2059 break;
2060 case spv::OpCapability:
2061 if (insn.word(1) == spv::CapabilityCooperativeMatrixNV) {
2062 seen_coopmat_capability = true;
2063
2064 if (!(pStage->stage & phys_dev_ext_props.cooperative_matrix_props.cooperativeMatrixSupportedStages)) {
2065 skip |=
Mark Lobodzinski93a1fa72019-04-19 12:12:25 -06002066 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
Jeff Bolze4356752019-03-07 11:23:46 -06002067 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_CooperativeMatrixSupportedStages,
2068 "OpTypeCooperativeMatrixNV used in shader stage not in cooperativeMatrixSupportedStages (= %u)",
2069 phys_dev_ext_props.cooperative_matrix_props.cooperativeMatrixSupportedStages);
2070 }
2071 }
2072 break;
2073 case spv::OpMemoryModel:
2074 // If the capability isn't enabled, don't bother with the rest of this function.
2075 // OpMemoryModel is the first required instruction after all OpCapability instructions.
2076 if (!seen_coopmat_capability) {
2077 return skip;
2078 }
2079 break;
2080 case spv::OpTypeCooperativeMatrixNV: {
2081 CoopMatType M;
2082 M.Init(insn.word(1), src, pStage, id_to_spec_id);
2083
2084 if (M.all_constant) {
2085 // Validate that the type parameters are all supported for one of the
2086 // operands of a cooperative matrix property.
2087 bool valid = false;
2088 for (unsigned i = 0; i < cooperative_matrix_properties.size(); ++i) {
2089 if (cooperative_matrix_properties[i].AType == M.component_type &&
2090 cooperative_matrix_properties[i].MSize == M.rows && cooperative_matrix_properties[i].KSize == M.cols &&
2091 cooperative_matrix_properties[i].scope == M.scope) {
2092 valid = true;
2093 break;
2094 }
2095 if (cooperative_matrix_properties[i].BType == M.component_type &&
2096 cooperative_matrix_properties[i].KSize == M.rows && cooperative_matrix_properties[i].NSize == M.cols &&
2097 cooperative_matrix_properties[i].scope == M.scope) {
2098 valid = true;
2099 break;
2100 }
2101 if (cooperative_matrix_properties[i].CType == M.component_type &&
2102 cooperative_matrix_properties[i].MSize == M.rows && cooperative_matrix_properties[i].NSize == M.cols &&
2103 cooperative_matrix_properties[i].scope == M.scope) {
2104 valid = true;
2105 break;
2106 }
2107 if (cooperative_matrix_properties[i].DType == M.component_type &&
2108 cooperative_matrix_properties[i].MSize == M.rows && cooperative_matrix_properties[i].NSize == M.cols &&
2109 cooperative_matrix_properties[i].scope == M.scope) {
2110 valid = true;
2111 break;
2112 }
2113 }
2114 if (!valid) {
Mark Lobodzinski93a1fa72019-04-19 12:12:25 -06002115 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
Jeff Bolze4356752019-03-07 11:23:46 -06002116 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_CooperativeMatrixType,
2117 "OpTypeCooperativeMatrixNV (result id = %u) operands don't match a supported matrix type",
2118 insn.word(1));
2119 }
2120 }
2121 break;
2122 }
2123 case spv::OpCooperativeMatrixMulAddNV: {
2124 CoopMatType A, B, C, D;
2125 if (id_to_type_id.find(insn.word(2)) == id_to_type_id.end() ||
2126 id_to_type_id.find(insn.word(3)) == id_to_type_id.end() ||
2127 id_to_type_id.find(insn.word(4)) == id_to_type_id.end() ||
2128 id_to_type_id.find(insn.word(5)) == id_to_type_id.end()) {
Mike Schuchardte48dc142019-04-18 09:12:03 -07002129 // Couldn't find type of matrix
2130 assert(false);
Jeff Bolze4356752019-03-07 11:23:46 -06002131 break;
2132 }
2133 D.Init(id_to_type_id[insn.word(2)], src, pStage, id_to_spec_id);
2134 A.Init(id_to_type_id[insn.word(3)], src, pStage, id_to_spec_id);
2135 B.Init(id_to_type_id[insn.word(4)], src, pStage, id_to_spec_id);
2136 C.Init(id_to_type_id[insn.word(5)], src, pStage, id_to_spec_id);
2137
2138 if (A.all_constant && B.all_constant && C.all_constant && D.all_constant) {
2139 // Validate that the type parameters are all supported for the same
2140 // cooperative matrix property.
2141 bool valid = false;
2142 for (unsigned i = 0; i < cooperative_matrix_properties.size(); ++i) {
2143 if (cooperative_matrix_properties[i].AType == A.component_type &&
2144 cooperative_matrix_properties[i].MSize == A.rows && cooperative_matrix_properties[i].KSize == A.cols &&
2145 cooperative_matrix_properties[i].scope == A.scope &&
2146
2147 cooperative_matrix_properties[i].BType == B.component_type &&
2148 cooperative_matrix_properties[i].KSize == B.rows && cooperative_matrix_properties[i].NSize == B.cols &&
2149 cooperative_matrix_properties[i].scope == B.scope &&
2150
2151 cooperative_matrix_properties[i].CType == C.component_type &&
2152 cooperative_matrix_properties[i].MSize == C.rows && cooperative_matrix_properties[i].NSize == C.cols &&
2153 cooperative_matrix_properties[i].scope == C.scope &&
2154
2155 cooperative_matrix_properties[i].DType == D.component_type &&
2156 cooperative_matrix_properties[i].MSize == D.rows && cooperative_matrix_properties[i].NSize == D.cols &&
2157 cooperative_matrix_properties[i].scope == D.scope) {
2158 valid = true;
2159 break;
2160 }
2161 }
2162 if (!valid) {
Mark Lobodzinski93a1fa72019-04-19 12:12:25 -06002163 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
Jeff Bolze4356752019-03-07 11:23:46 -06002164 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_CooperativeMatrixMulAdd,
2165 "OpCooperativeMatrixMulAddNV (result id = %u) operands don't match a supported matrix "
2166 "VkCooperativeMatrixPropertiesNV",
2167 insn.word(2));
2168 }
2169 }
2170 break;
2171 }
2172 default:
2173 break;
2174 }
2175 }
2176
2177 return skip;
2178}
2179
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002180bool CoreChecks::ValidateExecutionModes(SHADER_MODULE_STATE const *src, spirv_inst_iter entrypoint) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002181 auto entrypoint_id = entrypoint.word(2);
2182
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01002183 // The first denorm execution mode encountered, along with its bit width.
2184 // Used to check if SeparateDenormSettings is respected.
2185 std::pair<spv::ExecutionMode, uint32_t> first_denorm_execution_mode = std::make_pair(spv::ExecutionModeMax, 0);
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002186
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01002187 // The first rounding mode encountered, along with its bit width.
2188 // Used to check if SeparateRoundingModeSettings is respected.
2189 std::pair<spv::ExecutionMode, uint32_t> first_rounding_mode = std::make_pair(spv::ExecutionModeMax, 0);
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002190
2191 bool skip = false;
2192
Jeff Bolze9ee3d82019-05-29 13:45:13 -05002193 uint32_t verticesOut = 0;
2194 uint32_t invocations = 0;
2195
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002196 for (auto insn : *src) {
2197 if (insn.opcode() == spv::OpExecutionMode && insn.word(1) == entrypoint_id) {
2198 auto mode = insn.word(2);
2199 switch (mode) {
2200 case spv::ExecutionModeSignedZeroInfNanPreserve: {
2201 auto bit_width = insn.word(3);
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002202 if ((bit_width == 16 && !enabled_features.float_controls.shaderSignedZeroInfNanPreserveFloat16) ||
2203 (bit_width == 32 && !enabled_features.float_controls.shaderSignedZeroInfNanPreserveFloat32) ||
2204 (bit_width == 64 && !enabled_features.float_controls.shaderSignedZeroInfNanPreserveFloat64)) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002205 skip |=
2206 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2207 kVUID_Core_Shader_FeatureNotEnabled,
2208 "Shader requires SignedZeroInfNanPreserve for bit width %d but it is not enabled on the device",
2209 bit_width);
2210 }
2211 break;
2212 }
2213
2214 case spv::ExecutionModeDenormPreserve: {
2215 auto bit_width = insn.word(3);
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002216 if ((bit_width == 16 && !enabled_features.float_controls.shaderDenormPreserveFloat16) ||
2217 (bit_width == 32 && !enabled_features.float_controls.shaderDenormPreserveFloat32) ||
2218 (bit_width == 64 && !enabled_features.float_controls.shaderDenormPreserveFloat64)) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002219 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2220 kVUID_Core_Shader_FeatureNotEnabled,
2221 "Shader requires DenormPreserve for bit width %d but it is not enabled on the device",
2222 bit_width);
2223 }
2224
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01002225 if (first_denorm_execution_mode.first == spv::ExecutionModeMax) {
2226 // Register the first denorm execution mode found
2227 first_denorm_execution_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
2228 } else if (first_denorm_execution_mode.first != mode && first_denorm_execution_mode.second != bit_width &&
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002229 !enabled_features.float_controls.separateDenormSettings) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002230 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2231 kVUID_Core_Shader_FeatureNotEnabled,
2232 "Shader uses separate denorm execution modes for different bit widths but "
2233 "SeparateDenormSettings is not enabled on the device");
2234 }
2235 break;
2236 }
2237
2238 case spv::ExecutionModeDenormFlushToZero: {
2239 auto bit_width = insn.word(3);
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002240 if ((bit_width == 16 && !enabled_features.float_controls.shaderDenormFlushToZeroFloat16) ||
2241 (bit_width == 32 && !enabled_features.float_controls.shaderDenormFlushToZeroFloat32) ||
2242 (bit_width == 64 && !enabled_features.float_controls.shaderDenormFlushToZeroFloat64)) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002243 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2244 kVUID_Core_Shader_FeatureNotEnabled,
2245 "Shader requires DenormFlushToZero for bit width %d but it is not enabled on the device",
2246 bit_width);
2247 }
2248
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01002249 if (first_denorm_execution_mode.first == spv::ExecutionModeMax) {
2250 // Register the first denorm execution mode found
2251 first_denorm_execution_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
2252 } else if (first_denorm_execution_mode.first != mode && first_denorm_execution_mode.second != bit_width &&
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002253 !enabled_features.float_controls.separateDenormSettings) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002254 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2255 kVUID_Core_Shader_FeatureNotEnabled,
2256 "Shader uses separate denorm execution modes for different bit widths but "
2257 "SeparateDenormSettings is not enabled on the device");
2258 }
2259 break;
2260 }
2261
2262 case spv::ExecutionModeRoundingModeRTE: {
2263 auto bit_width = insn.word(3);
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002264 if ((bit_width == 16 && !enabled_features.float_controls.shaderRoundingModeRTEFloat16) ||
2265 (bit_width == 32 && !enabled_features.float_controls.shaderRoundingModeRTEFloat32) ||
2266 (bit_width == 64 && !enabled_features.float_controls.shaderRoundingModeRTEFloat64)) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002267 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2268 kVUID_Core_Shader_FeatureNotEnabled,
2269 "Shader requires RoundingModeRTE for bit width %d but it is not enabled on the device",
2270 bit_width);
2271 }
2272
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01002273 if (first_rounding_mode.first == spv::ExecutionModeMax) {
2274 // Register the first rounding mode found
2275 first_rounding_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
2276 } else if (first_rounding_mode.first != mode && first_rounding_mode.second != bit_width &&
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002277 !enabled_features.float_controls.separateRoundingModeSettings) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002278 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2279 kVUID_Core_Shader_FeatureNotEnabled,
2280 "Shader uses separate rounding modes for different bit widths but "
2281 "SeparateRoundingModeSettings is not enabled on the device");
2282 }
2283 break;
2284 }
2285
2286 case spv::ExecutionModeRoundingModeRTZ: {
2287 auto bit_width = insn.word(3);
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002288 if ((bit_width == 16 && !enabled_features.float_controls.shaderRoundingModeRTZFloat16) ||
2289 (bit_width == 32 && !enabled_features.float_controls.shaderRoundingModeRTZFloat32) ||
2290 (bit_width == 64 && !enabled_features.float_controls.shaderRoundingModeRTZFloat64)) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002291 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2292 kVUID_Core_Shader_FeatureNotEnabled,
2293 "Shader requires RoundingModeRTZ for bit width %d but it is not enabled on the device",
2294 bit_width);
2295 }
2296
Attilio Provenzanof6c0e852019-04-09 11:01:18 +01002297 if (first_rounding_mode.first == spv::ExecutionModeMax) {
2298 // Register the first rounding mode found
2299 first_rounding_mode = std::make_pair(static_cast<spv::ExecutionMode>(mode), bit_width);
2300 } else if (first_rounding_mode.first != mode && first_rounding_mode.second != bit_width &&
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002301 !enabled_features.float_controls.separateRoundingModeSettings) {
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002302 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2303 kVUID_Core_Shader_FeatureNotEnabled,
2304 "Shader uses separate rounding modes for different bit widths but "
2305 "SeparateRoundingModeSettings is not enabled on the device");
2306 }
2307 break;
2308 }
Jeff Bolze9ee3d82019-05-29 13:45:13 -05002309
2310 case spv::ExecutionModeOutputVertices: {
2311 verticesOut = insn.word(3);
2312 break;
2313 }
2314
2315 case spv::ExecutionModeInvocations: {
2316 invocations = insn.word(3);
2317 break;
2318 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002319 }
2320 }
2321 }
2322
Jeff Bolze9ee3d82019-05-29 13:45:13 -05002323 if (entrypoint.word(1) == spv::ExecutionModelGeometry) {
2324 if (verticesOut == 0 || verticesOut > phys_dev_props.limits.maxGeometryOutputVertices) {
2325 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2326 "VUID-VkPipelineShaderStageCreateInfo-stage-00714",
2327 "Geometry shader entry point must have an OpExecutionMode instruction that "
2328 "specifies a maximum output vertex count that is greater than 0 and less "
2329 "than or equal to maxGeometryOutputVertices. "
2330 "OutputVertices=%d, maxGeometryOutputVertices=%d",
2331 verticesOut, phys_dev_props.limits.maxGeometryOutputVertices);
2332 }
2333
2334 if (invocations == 0 || invocations > phys_dev_props.limits.maxGeometryShaderInvocations) {
2335 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
2336 "VUID-VkPipelineShaderStageCreateInfo-stage-00715",
2337 "Geometry shader entry point must have an OpExecutionMode instruction that "
2338 "specifies an invocation count that is greater than 0 and less "
2339 "than or equal to maxGeometryShaderInvocations. "
2340 "Invocations=%d, maxGeometryShaderInvocations=%d",
2341 invocations, phys_dev_props.limits.maxGeometryShaderInvocations);
2342 }
2343 }
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002344 return skip;
2345}
2346
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002347static uint32_t DescriptorTypeToReqs(SHADER_MODULE_STATE const *module, uint32_t type_id) {
Chris Forbes47567b72017-06-09 12:09:45 -07002348 auto type = module->get_def(type_id);
2349
2350 while (true) {
2351 switch (type.opcode()) {
2352 case spv::OpTypeArray:
Chris Forbes062f1222018-08-21 15:34:15 -07002353 case spv::OpTypeRuntimeArray:
Chris Forbes47567b72017-06-09 12:09:45 -07002354 case spv::OpTypeSampledImage:
2355 type = module->get_def(type.word(2));
2356 break;
2357 case spv::OpTypePointer:
2358 type = module->get_def(type.word(3));
2359 break;
2360 case spv::OpTypeImage: {
2361 auto dim = type.word(3);
2362 auto arrayed = type.word(5);
2363 auto msaa = type.word(6);
2364
Chris Forbes74ba2232018-08-27 15:19:27 -07002365 uint32_t bits = 0;
2366 switch (GetFundamentalType(module, type.word(2))) {
2367 case FORMAT_TYPE_FLOAT:
2368 bits = DESCRIPTOR_REQ_COMPONENT_TYPE_FLOAT;
2369 break;
2370 case FORMAT_TYPE_UINT:
2371 bits = DESCRIPTOR_REQ_COMPONENT_TYPE_UINT;
2372 break;
2373 case FORMAT_TYPE_SINT:
2374 bits = DESCRIPTOR_REQ_COMPONENT_TYPE_SINT;
2375 break;
2376 default:
2377 break;
2378 }
2379
Chris Forbes47567b72017-06-09 12:09:45 -07002380 switch (dim) {
2381 case spv::Dim1D:
Chris Forbes74ba2232018-08-27 15:19:27 -07002382 bits |= arrayed ? DESCRIPTOR_REQ_VIEW_TYPE_1D_ARRAY : DESCRIPTOR_REQ_VIEW_TYPE_1D;
2383 return bits;
Chris Forbes47567b72017-06-09 12:09:45 -07002384 case spv::Dim2D:
Chris Forbes74ba2232018-08-27 15:19:27 -07002385 bits |= msaa ? DESCRIPTOR_REQ_MULTI_SAMPLE : DESCRIPTOR_REQ_SINGLE_SAMPLE;
2386 bits |= arrayed ? DESCRIPTOR_REQ_VIEW_TYPE_2D_ARRAY : DESCRIPTOR_REQ_VIEW_TYPE_2D;
2387 return bits;
Chris Forbes47567b72017-06-09 12:09:45 -07002388 case spv::Dim3D:
Chris Forbes74ba2232018-08-27 15:19:27 -07002389 bits |= DESCRIPTOR_REQ_VIEW_TYPE_3D;
2390 return bits;
Chris Forbes47567b72017-06-09 12:09:45 -07002391 case spv::DimCube:
Chris Forbes74ba2232018-08-27 15:19:27 -07002392 bits |= arrayed ? DESCRIPTOR_REQ_VIEW_TYPE_CUBE_ARRAY : DESCRIPTOR_REQ_VIEW_TYPE_CUBE;
2393 return bits;
Chris Forbes47567b72017-06-09 12:09:45 -07002394 case spv::DimSubpassData:
Chris Forbes74ba2232018-08-27 15:19:27 -07002395 bits |= msaa ? DESCRIPTOR_REQ_MULTI_SAMPLE : DESCRIPTOR_REQ_SINGLE_SAMPLE;
2396 return bits;
Chris Forbes47567b72017-06-09 12:09:45 -07002397 default: // buffer, etc.
Chris Forbes74ba2232018-08-27 15:19:27 -07002398 return bits;
Chris Forbes47567b72017-06-09 12:09:45 -07002399 }
2400 }
2401 default:
2402 return 0;
2403 }
2404 }
2405}
2406
2407// For given pipelineLayout verify that the set_layout_node at slot.first
2408// has the requested binding at slot.second and return ptr to that binding
Mark Lobodzinskica6ebe32019-04-25 11:43:37 -06002409static VkDescriptorSetLayoutBinding const *GetDescriptorBinding(PIPELINE_LAYOUT_STATE const *pipelineLayout,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002410 descriptor_slot_t slot) {
Chris Forbes47567b72017-06-09 12:09:45 -07002411 if (!pipelineLayout) return nullptr;
2412
2413 if (slot.first >= pipelineLayout->set_layouts.size()) return nullptr;
2414
2415 return pipelineLayout->set_layouts[slot.first]->GetDescriptorSetLayoutBindingPtrFromBinding(slot.second);
2416}
2417
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002418static bool FindLocalSize(SHADER_MODULE_STATE const *src, uint32_t &local_size_x, uint32_t &local_size_y, uint32_t &local_size_z) {
Locke1ec6d952019-04-02 11:57:21 -06002419 for (auto insn : *src) {
2420 if (insn.opcode() == spv::OpEntryPoint) {
2421 auto executionModel = insn.word(1);
2422 auto entrypointStageBits = ExecutionModelToShaderStageFlagBits(executionModel);
2423 if (entrypointStageBits == VK_SHADER_STAGE_COMPUTE_BIT) {
2424 auto entrypoint_id = insn.word(2);
2425 for (auto insn1 : *src) {
2426 if (insn1.opcode() == spv::OpExecutionMode && insn1.word(1) == entrypoint_id &&
2427 insn1.word(2) == spv::ExecutionModeLocalSize) {
2428 local_size_x = insn1.word(3);
2429 local_size_y = insn1.word(4);
2430 local_size_z = insn1.word(5);
2431 return true;
2432 }
2433 }
2434 }
2435 }
2436 }
2437 return false;
2438}
2439
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002440static void ProcessExecutionModes(SHADER_MODULE_STATE const *src, spirv_inst_iter entrypoint, PIPELINE_STATE *pipeline) {
Jeff Bolz105d6492018-09-29 15:46:44 -05002441 auto entrypoint_id = entrypoint.word(2);
Chris Forbes0771b672018-03-22 21:13:46 -07002442 bool is_point_mode = false;
2443
2444 for (auto insn : *src) {
2445 if (insn.opcode() == spv::OpExecutionMode && insn.word(1) == entrypoint_id) {
2446 switch (insn.word(2)) {
2447 case spv::ExecutionModePointMode:
2448 // In tessellation shaders, PointMode is separate and trumps the tessellation topology.
2449 is_point_mode = true;
2450 break;
2451
2452 case spv::ExecutionModeOutputPoints:
2453 pipeline->topology_at_rasterizer = VK_PRIMITIVE_TOPOLOGY_POINT_LIST;
2454 break;
2455
2456 case spv::ExecutionModeIsolines:
2457 case spv::ExecutionModeOutputLineStrip:
2458 pipeline->topology_at_rasterizer = VK_PRIMITIVE_TOPOLOGY_LINE_STRIP;
2459 break;
2460
2461 case spv::ExecutionModeTriangles:
2462 case spv::ExecutionModeQuads:
2463 case spv::ExecutionModeOutputTriangleStrip:
2464 pipeline->topology_at_rasterizer = VK_PRIMITIVE_TOPOLOGY_TRIANGLE_STRIP;
2465 break;
2466 }
2467 }
2468 }
2469
2470 if (is_point_mode) pipeline->topology_at_rasterizer = VK_PRIMITIVE_TOPOLOGY_POINT_LIST;
2471}
2472
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002473// If PointList topology is specified in the pipeline, verify that a shader geometry stage writes PointSize
2474// o If there is only a vertex shader : gl_PointSize must be written when using points
2475// o If there is a geometry or tessellation shader:
2476// - If shaderTessellationAndGeometryPointSize feature is enabled:
2477// * gl_PointSize must be written in the final geometry stage
2478// - If shaderTessellationAndGeometryPointSize feature is disabled:
2479// * gl_PointSize must NOT be written and a default of 1.0 is assumed
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002480bool CoreChecks::ValidatePointListShaderState(const PIPELINE_STATE *pipeline, SHADER_MODULE_STATE const *src,
2481 spirv_inst_iter entrypoint, VkShaderStageFlagBits stage) {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002482 if (pipeline->topology_at_rasterizer != VK_PRIMITIVE_TOPOLOGY_POINT_LIST) {
2483 return false;
2484 }
2485
2486 bool pointsize_written = false;
2487 bool skip = false;
2488
2489 // Search for PointSize built-in decorations
2490 std::vector<uint32_t> pointsize_builtin_offsets;
2491 spirv_inst_iter insn = entrypoint;
2492 while (!pointsize_written && (insn.opcode() != spv::OpFunction)) {
2493 if (insn.opcode() == spv::OpMemberDecorate) {
2494 if (insn.word(3) == spv::DecorationBuiltIn) {
2495 if (insn.word(4) == spv::BuiltInPointSize) {
2496 pointsize_written = IsPointSizeWritten(src, insn, entrypoint);
2497 }
2498 }
2499 } else if (insn.opcode() == spv::OpDecorate) {
2500 if (insn.word(2) == spv::DecorationBuiltIn) {
2501 if (insn.word(3) == spv::BuiltInPointSize) {
2502 pointsize_written = IsPointSizeWritten(src, insn, entrypoint);
2503 }
2504 }
2505 }
2506
2507 insn++;
2508 }
2509
2510 if ((stage == VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT || stage == VK_SHADER_STAGE_GEOMETRY_BIT) &&
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002511 !enabled_features.core.shaderTessellationAndGeometryPointSize) {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002512 if (pointsize_written) {
Mark Lobodzinski93a1fa72019-04-19 12:12:25 -06002513 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002514 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_PointSizeBuiltInOverSpecified,
2515 "Pipeline topology is set to POINT_LIST and geometry or tessellation shaders write PointSize which "
2516 "is prohibited when the shaderTessellationAndGeometryPointSize feature is not enabled.");
2517 }
2518 } else if (!pointsize_written) {
2519 skip |=
Mark Lobodzinski93a1fa72019-04-19 12:12:25 -06002520 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_PIPELINE_EXT,
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002521 HandleToUint64(pipeline->pipeline), kVUID_Core_Shader_MissingPointSizeBuiltIn,
2522 "Pipeline topology is set to POINT_LIST, but PointSize is not written to in the shader corresponding to %s.",
2523 string_VkShaderStageFlagBits(stage));
2524 }
2525 return skip;
2526}
2527
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002528bool CoreChecks::ValidatePipelineShaderStage(VkPipelineShaderStageCreateInfo const *pStage, PIPELINE_STATE *pipeline,
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002529 SHADER_MODULE_STATE const **out_module, spirv_inst_iter *out_entrypoint,
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002530 bool check_point_size) {
Chris Forbes47567b72017-06-09 12:09:45 -07002531 bool skip = false;
Mark Lobodzinski9e9da292019-03-06 16:19:55 -07002532 auto module = *out_module = GetShaderModuleState(pStage->module);
Chris Forbes47567b72017-06-09 12:09:45 -07002533
2534 if (!module->has_valid_spirv) return false;
2535
2536 // Find the entrypoint
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002537 auto entrypoint = *out_entrypoint = FindEntrypoint(module, pStage->pName, pStage->stage);
Chris Forbes47567b72017-06-09 12:09:45 -07002538 if (entrypoint == module->end()) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06002539 if (log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton78d09922018-05-17 15:48:45 -06002540 "VUID-VkPipelineShaderStageCreateInfo-pName-00707", "No entrypoint found named `%s` for stage %s..",
2541 pStage->pName, string_VkShaderStageFlagBits(pStage->stage))) {
Chris Forbes47567b72017-06-09 12:09:45 -07002542 return true; // no point continuing beyond here, any analysis is just going to be garbage.
2543 }
2544 }
2545
Chris Forbes47567b72017-06-09 12:09:45 -07002546 // Mark accessible ids
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002547 auto accessible_ids = MarkAccessibleIds(module, entrypoint);
2548 ProcessExecutionModes(module, entrypoint, pipeline);
Chris Forbes47567b72017-06-09 12:09:45 -07002549
2550 // Validate descriptor set layout against what the entrypoint actually uses
Chris Forbes8af24522018-03-07 11:37:45 -08002551 bool has_writable_descriptor = false;
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002552 auto descriptor_uses = CollectInterfaceByDescriptorSlot(report_data, module, accessible_ids, &has_writable_descriptor);
Chris Forbes47567b72017-06-09 12:09:45 -07002553
Chris Forbes349b3132018-03-07 11:38:08 -08002554 // Validate shader capabilities against enabled device features
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002555 skip |= ValidateShaderCapabilities(module, pStage->stage, has_writable_descriptor);
Jeff Bolze9ee3d82019-05-29 13:45:13 -05002556 skip |= ValidateShaderStageInputOutputLimits(module, pStage, pipeline, entrypoint);
Attilio Provenzanoc5d50102019-03-25 17:40:37 +00002557 skip |= ValidateExecutionModes(module, entrypoint);
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002558 skip |= ValidateSpecializationOffsets(report_data, pStage);
2559 skip |= ValidatePushConstantUsage(report_data, pipeline->pipeline_layout.push_constant_ranges.get(), module, accessible_ids,
2560 pStage->stage);
Jeff Bolze54ae892018-09-08 12:16:29 -05002561 if (check_point_size && !pipeline->graphicsPipelineCI.pRasterizationState->rasterizerDiscardEnable) {
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002562 skip |= ValidatePointListShaderState(pipeline, module, entrypoint, pStage->stage);
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002563 }
Jeff Bolze4356752019-03-07 11:23:46 -06002564 skip |= ValidateCooperativeMatrix(module, pStage, pipeline);
Chris Forbes47567b72017-06-09 12:09:45 -07002565
2566 // Validate descriptor use
2567 for (auto use : descriptor_uses) {
2568 // While validating shaders capture which slots are used by the pipeline
2569 auto &reqs = pipeline->active_slots[use.first.first][use.first.second];
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002570 reqs = descriptor_req(reqs | DescriptorTypeToReqs(module, use.second.type_id));
Chris Forbes47567b72017-06-09 12:09:45 -07002571
2572 // Verify given pipelineLayout has requested setLayout with requested binding
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002573 const auto &binding = GetDescriptorBinding(&pipeline->pipeline_layout, use.first);
Chris Forbes47567b72017-06-09 12:09:45 -07002574 unsigned required_descriptor_count;
Jeff Bolze54ae892018-09-08 12:16:29 -05002575 std::set<uint32_t> descriptor_types = TypeToDescriptorTypeSet(module, use.second.type_id, required_descriptor_count);
Chris Forbes47567b72017-06-09 12:09:45 -07002576
2577 if (!binding) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06002578 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06002579 kVUID_Core_Shader_MissingDescriptor,
Chris Forbes73c00bf2018-06-22 16:28:06 -07002580 "Shader uses descriptor slot %u.%u (expected `%s`) but not declared in pipeline layout",
Jeff Bolze54ae892018-09-08 12:16:29 -05002581 use.first.first, use.first.second, string_descriptorTypes(descriptor_types).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -07002582 } else if (~binding->stageFlags & pStage->stage) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06002583 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_DEVICE_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06002584 kVUID_Core_Shader_DescriptorNotAccessibleFromStage,
Chris Forbes73c00bf2018-06-22 16:28:06 -07002585 "Shader uses descriptor slot %u.%u but descriptor not accessible from stage %s", use.first.first,
2586 use.first.second, string_VkShaderStageFlagBits(pStage->stage));
Jeff Bolze54ae892018-09-08 12:16:29 -05002587 } else if (descriptor_types.find(binding->descriptorType) == descriptor_types.end()) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06002588 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06002589 kVUID_Core_Shader_DescriptorTypeMismatch,
Chris Forbes73c00bf2018-06-22 16:28:06 -07002590 "Type mismatch on descriptor slot %u.%u (expected `%s`) but descriptor of type %s", use.first.first,
Jeff Bolze54ae892018-09-08 12:16:29 -05002591 use.first.second, string_descriptorTypes(descriptor_types).c_str(),
Chris Forbes47567b72017-06-09 12:09:45 -07002592 string_VkDescriptorType(binding->descriptorType));
2593 } else if (binding->descriptorCount < required_descriptor_count) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06002594 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06002595 kVUID_Core_Shader_DescriptorTypeMismatch,
Chris Forbes73c00bf2018-06-22 16:28:06 -07002596 "Shader expects at least %u descriptors for binding %u.%u but only %u provided",
2597 required_descriptor_count, use.first.first, use.first.second, binding->descriptorCount);
Chris Forbes47567b72017-06-09 12:09:45 -07002598 }
2599 }
2600
2601 // Validate use of input attachments against subpass structure
2602 if (pStage->stage == VK_SHADER_STAGE_FRAGMENT_BIT) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002603 auto input_attachment_uses = CollectInterfaceByInputAttachmentIndex(module, accessible_ids);
Chris Forbes47567b72017-06-09 12:09:45 -07002604
Petr Krause91f7a12017-12-14 20:57:36 +01002605 auto rpci = pipeline->rp_state->createInfo.ptr();
Chris Forbes47567b72017-06-09 12:09:45 -07002606 auto subpass = pipeline->graphicsPipelineCI.subpass;
2607
2608 for (auto use : input_attachment_uses) {
2609 auto input_attachments = rpci->pSubpasses[subpass].pInputAttachments;
2610 auto index = (input_attachments && use.first < rpci->pSubpasses[subpass].inputAttachmentCount)
Dave Houltona9df0ce2018-02-07 10:51:23 -07002611 ? input_attachments[use.first].attachment
2612 : VK_ATTACHMENT_UNUSED;
Chris Forbes47567b72017-06-09 12:09:45 -07002613
2614 if (index == VK_ATTACHMENT_UNUSED) {
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06002615 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06002616 kVUID_Core_Shader_MissingInputAttachment,
Chris Forbes47567b72017-06-09 12:09:45 -07002617 "Shader consumes input attachment index %d but not provided in subpass", use.first);
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002618 } else if (!(GetFormatType(rpci->pAttachments[index].format) & GetFundamentalType(module, use.second.type_id))) {
Chris Forbes47567b72017-06-09 12:09:45 -07002619 skip |=
Mark Lobodzinskib1fd9d12018-03-30 14:26:00 -06002620 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton51653902018-06-22 17:32:13 -06002621 kVUID_Core_Shader_InputAttachmentTypeMismatch,
Chris Forbes47567b72017-06-09 12:09:45 -07002622 "Subpass input attachment %u format of %s does not match type used in shader `%s`", use.first,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002623 string_VkFormat(rpci->pAttachments[index].format), DescribeType(module, use.second.type_id).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -07002624 }
2625 }
2626 }
Lockeaa8fdc02019-04-02 11:59:20 -06002627 if (pStage->stage == VK_SHADER_STAGE_COMPUTE_BIT) {
2628 skip |= ValidateComputeWorkGroupSizes(module);
2629 }
Chris Forbes47567b72017-06-09 12:09:45 -07002630 return skip;
2631}
2632
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002633static bool ValidateInterfaceBetweenStages(debug_report_data const *report_data, SHADER_MODULE_STATE const *producer,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002634 spirv_inst_iter producer_entrypoint, shader_stage_attributes const *producer_stage,
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002635 SHADER_MODULE_STATE const *consumer, spirv_inst_iter consumer_entrypoint,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002636 shader_stage_attributes const *consumer_stage) {
Chris Forbes47567b72017-06-09 12:09:45 -07002637 bool skip = false;
2638
2639 auto outputs =
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002640 CollectInterfaceByLocation(producer, producer_entrypoint, spv::StorageClassOutput, producer_stage->arrayed_output);
2641 auto inputs = CollectInterfaceByLocation(consumer, consumer_entrypoint, spv::StorageClassInput, consumer_stage->arrayed_input);
Chris Forbes47567b72017-06-09 12:09:45 -07002642
2643 auto a_it = outputs.begin();
2644 auto b_it = inputs.begin();
2645
2646 // Maps sorted by key (location); walk them together to find mismatches
2647 while ((outputs.size() > 0 && a_it != outputs.end()) || (inputs.size() && b_it != inputs.end())) {
2648 bool a_at_end = outputs.size() == 0 || a_it == outputs.end();
2649 bool b_at_end = inputs.size() == 0 || b_it == inputs.end();
2650 auto a_first = a_at_end ? std::make_pair(0u, 0u) : a_it->first;
2651 auto b_first = b_at_end ? std::make_pair(0u, 0u) : b_it->first;
2652
2653 if (b_at_end || ((!a_at_end) && (a_first < b_first))) {
Mark Young4e919b22018-05-21 15:53:59 -06002654 skip |= log_msg(report_data, VK_DEBUG_REPORT_PERFORMANCE_WARNING_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -06002655 HandleToUint64(producer->vk_shader_module), kVUID_Core_Shader_OutputNotConsumed,
Mark Young4e919b22018-05-21 15:53:59 -06002656 "%s writes to output location %u.%u which is not consumed by %s", producer_stage->name, a_first.first,
2657 a_first.second, consumer_stage->name);
Chris Forbes47567b72017-06-09 12:09:45 -07002658 a_it++;
2659 } else if (a_at_end || a_first > b_first) {
Mark Young4e919b22018-05-21 15:53:59 -06002660 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -06002661 HandleToUint64(consumer->vk_shader_module), kVUID_Core_Shader_InputNotProduced,
Mark Young4e919b22018-05-21 15:53:59 -06002662 "%s consumes input location %u.%u which is not written by %s", consumer_stage->name, b_first.first,
2663 b_first.second, producer_stage->name);
Chris Forbes47567b72017-06-09 12:09:45 -07002664 b_it++;
2665 } else {
2666 // subtleties of arrayed interfaces:
2667 // - if is_patch, then the member is not arrayed, even though the interface may be.
2668 // - if is_block_member, then the extra array level of an arrayed interface is not
2669 // expressed in the member type -- it's expressed in the block type.
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002670 if (!TypesMatch(producer, consumer, a_it->second.type_id, b_it->second.type_id,
2671 producer_stage->arrayed_output && !a_it->second.is_patch && !a_it->second.is_block_member,
2672 consumer_stage->arrayed_input && !b_it->second.is_patch && !b_it->second.is_block_member, true)) {
Mark Young4e919b22018-05-21 15:53:59 -06002673 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -06002674 HandleToUint64(producer->vk_shader_module), kVUID_Core_Shader_InterfaceTypeMismatch,
Mark Young4e919b22018-05-21 15:53:59 -06002675 "Type mismatch on location %u.%u: '%s' vs '%s'", a_first.first, a_first.second,
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002676 DescribeType(producer, a_it->second.type_id).c_str(),
2677 DescribeType(consumer, b_it->second.type_id).c_str());
Chris Forbes47567b72017-06-09 12:09:45 -07002678 }
2679 if (a_it->second.is_patch != b_it->second.is_patch) {
Mark Young4e919b22018-05-21 15:53:59 -06002680 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -06002681 HandleToUint64(producer->vk_shader_module), kVUID_Core_Shader_InterfaceTypeMismatch,
Dave Houltona9df0ce2018-02-07 10:51:23 -07002682 "Decoration mismatch on location %u.%u: is per-%s in %s stage but per-%s in %s stage",
Chris Forbes47567b72017-06-09 12:09:45 -07002683 a_first.first, a_first.second, a_it->second.is_patch ? "patch" : "vertex", producer_stage->name,
2684 b_it->second.is_patch ? "patch" : "vertex", consumer_stage->name);
2685 }
2686 if (a_it->second.is_relaxed_precision != b_it->second.is_relaxed_precision) {
Mark Young4e919b22018-05-21 15:53:59 -06002687 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
Dave Houlton51653902018-06-22 17:32:13 -06002688 HandleToUint64(producer->vk_shader_module), kVUID_Core_Shader_InterfaceTypeMismatch,
Chris Forbes47567b72017-06-09 12:09:45 -07002689 "Decoration mismatch on location %u.%u: %s and %s stages differ in precision", a_first.first,
2690 a_first.second, producer_stage->name, consumer_stage->name);
2691 }
2692 a_it++;
2693 b_it++;
2694 }
2695 }
2696
Ari Suonpaa696b3432019-03-11 14:02:57 +02002697 if (consumer_stage->stage != VK_SHADER_STAGE_FRAGMENT_BIT) {
2698 auto builtins_producer = CollectBuiltinBlockMembers(producer, producer_entrypoint, spv::StorageClassOutput);
2699 auto builtins_consumer = CollectBuiltinBlockMembers(consumer, consumer_entrypoint, spv::StorageClassInput);
2700
2701 if (!builtins_producer.empty() && !builtins_consumer.empty()) {
2702 if (builtins_producer.size() != builtins_consumer.size()) {
2703 skip |=
2704 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
2705 HandleToUint64(producer->vk_shader_module), kVUID_Core_Shader_InterfaceTypeMismatch,
2706 "Number of elements inside builtin block differ between stages (%s %d vs %s %d).", producer_stage->name,
2707 (int)builtins_producer.size(), consumer_stage->name, (int)builtins_consumer.size());
2708 } else {
2709 auto it_producer = builtins_producer.begin();
2710 auto it_consumer = builtins_consumer.begin();
2711 while (it_producer != builtins_producer.end() && it_consumer != builtins_consumer.end()) {
2712 if (*it_producer != *it_consumer) {
2713 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
2714 HandleToUint64(producer->vk_shader_module), kVUID_Core_Shader_InterfaceTypeMismatch,
2715 "Builtin variable inside block doesn't match between %s and %s.", producer_stage->name,
2716 consumer_stage->name);
2717 break;
2718 }
2719 it_producer++;
2720 it_consumer++;
2721 }
2722 }
2723 }
2724 }
2725
Chris Forbes47567b72017-06-09 12:09:45 -07002726 return skip;
2727}
2728
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002729static inline uint32_t DetermineFinalGeomStage(PIPELINE_STATE *pipeline, VkGraphicsPipelineCreateInfo *pCreateInfo) {
2730 uint32_t stage_mask = 0;
2731 if (pipeline->topology_at_rasterizer == VK_PRIMITIVE_TOPOLOGY_POINT_LIST) {
2732 for (uint32_t i = 0; i < pCreateInfo->stageCount; i++) {
2733 stage_mask |= pCreateInfo->pStages[i].stage;
2734 }
2735 // Determine which shader in which PointSize should be written (the final geometry stage)
Jeff Bolz105d6492018-09-29 15:46:44 -05002736 if (stage_mask & VK_SHADER_STAGE_MESH_BIT_NV) {
2737 stage_mask = VK_SHADER_STAGE_MESH_BIT_NV;
2738 } else if (stage_mask & VK_SHADER_STAGE_GEOMETRY_BIT) {
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002739 stage_mask = VK_SHADER_STAGE_GEOMETRY_BIT;
2740 } else if (stage_mask & VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT) {
2741 stage_mask = VK_SHADER_STAGE_TESSELLATION_EVALUATION_BIT;
2742 } else if (stage_mask & VK_SHADER_STAGE_VERTEX_BIT) {
2743 stage_mask = VK_SHADER_STAGE_VERTEX_BIT;
Mark Lobodzinski2c984cc2018-07-31 09:57:46 -06002744 }
2745 }
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002746 return stage_mask;
Mark Lobodzinski2c984cc2018-07-31 09:57:46 -06002747}
2748
Chris Forbes47567b72017-06-09 12:09:45 -07002749// Validate that the shaders used by the given pipeline and store the active_slots
2750// that are actually used by the pipeline into pPipeline->active_slots
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002751bool CoreChecks::ValidateAndCapturePipelineShaderState(PIPELINE_STATE *pipeline) {
Chris Forbesa400a8a2017-07-20 13:10:24 -07002752 auto pCreateInfo = pipeline->graphicsPipelineCI.ptr();
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002753 int vertex_stage = GetShaderStageId(VK_SHADER_STAGE_VERTEX_BIT);
2754 int fragment_stage = GetShaderStageId(VK_SHADER_STAGE_FRAGMENT_BIT);
Chris Forbes47567b72017-06-09 12:09:45 -07002755
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002756 SHADER_MODULE_STATE const *shaders[32];
Chris Forbes47567b72017-06-09 12:09:45 -07002757 memset(shaders, 0, sizeof(shaders));
Jeff Bolz7e35c392018-09-04 15:30:41 -05002758 spirv_inst_iter entrypoints[32];
Chris Forbes47567b72017-06-09 12:09:45 -07002759 memset(entrypoints, 0, sizeof(entrypoints));
2760 bool skip = false;
2761
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002762 uint32_t pointlist_stage_mask = DetermineFinalGeomStage(pipeline, pCreateInfo);
2763
Chris Forbes47567b72017-06-09 12:09:45 -07002764 for (uint32_t i = 0; i < pCreateInfo->stageCount; i++) {
2765 auto pStage = &pCreateInfo->pStages[i];
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002766 auto stage_id = GetShaderStageId(pStage->stage);
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002767 skip |= ValidatePipelineShaderStage(pStage, pipeline, &shaders[stage_id], &entrypoints[stage_id],
Mark Lobodzinski1b4a8ed2018-08-07 08:47:05 -06002768 (pointlist_stage_mask == pStage->stage));
Chris Forbes47567b72017-06-09 12:09:45 -07002769 }
2770
2771 // if the shader stages are no good individually, cross-stage validation is pointless.
2772 if (skip) return true;
2773
2774 auto vi = pCreateInfo->pVertexInputState;
2775
2776 if (vi) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002777 skip |= ValidateViConsistency(report_data, vi);
Chris Forbes47567b72017-06-09 12:09:45 -07002778 }
2779
2780 if (shaders[vertex_stage] && shaders[vertex_stage]->has_valid_spirv) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002781 skip |= ValidateViAgainstVsInputs(report_data, vi, shaders[vertex_stage], entrypoints[vertex_stage]);
Chris Forbes47567b72017-06-09 12:09:45 -07002782 }
2783
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002784 int producer = GetShaderStageId(VK_SHADER_STAGE_VERTEX_BIT);
2785 int consumer = GetShaderStageId(VK_SHADER_STAGE_TESSELLATION_CONTROL_BIT);
Chris Forbes47567b72017-06-09 12:09:45 -07002786
2787 while (!shaders[producer] && producer != fragment_stage) {
2788 producer++;
2789 consumer++;
2790 }
2791
2792 for (; producer != fragment_stage && consumer <= fragment_stage; consumer++) {
2793 assert(shaders[producer]);
Chris Forbesdbb43fc2018-02-16 16:59:23 -08002794 if (shaders[consumer]) {
2795 if (shaders[consumer]->has_valid_spirv && shaders[producer]->has_valid_spirv) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002796 skip |= ValidateInterfaceBetweenStages(report_data, shaders[producer], entrypoints[producer],
2797 &shader_stage_attribs[producer], shaders[consumer], entrypoints[consumer],
2798 &shader_stage_attribs[consumer]);
Chris Forbesdbb43fc2018-02-16 16:59:23 -08002799 }
Chris Forbes47567b72017-06-09 12:09:45 -07002800
2801 producer = consumer;
2802 }
2803 }
2804
2805 if (shaders[fragment_stage] && shaders[fragment_stage]->has_valid_spirv) {
Shannon McPhersonc06c33d2018-06-28 17:21:12 -06002806 skip |= ValidateFsOutputsAgainstRenderPass(report_data, shaders[fragment_stage], entrypoints[fragment_stage], pipeline,
2807 pCreateInfo->subpass);
Chris Forbes47567b72017-06-09 12:09:45 -07002808 }
2809
2810 return skip;
2811}
2812
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002813bool CoreChecks::ValidateComputePipeline(PIPELINE_STATE *pipeline) {
Chris Forbesa400a8a2017-07-20 13:10:24 -07002814 auto pCreateInfo = pipeline->computePipelineCI.ptr();
Chris Forbes47567b72017-06-09 12:09:45 -07002815
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002816 SHADER_MODULE_STATE const *module;
Chris Forbes47567b72017-06-09 12:09:45 -07002817 spirv_inst_iter entrypoint;
2818
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002819 return ValidatePipelineShaderStage(&pCreateInfo->stage, pipeline, &module, &entrypoint, false);
Chris Forbes47567b72017-06-09 12:09:45 -07002820}
Chris Forbes4ae55b32017-06-09 14:42:56 -07002821
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002822bool CoreChecks::ValidateRayTracingPipelineNV(PIPELINE_STATE *pipeline) {
Jeff Bolzfbe51582018-09-13 10:01:35 -05002823 auto pCreateInfo = pipeline->raytracingPipelineCI.ptr();
2824
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002825 SHADER_MODULE_STATE const *module;
Jeff Bolzfbe51582018-09-13 10:01:35 -05002826 spirv_inst_iter entrypoint;
2827
Mark Lobodzinski518eadc2019-03-09 12:07:30 -07002828 return ValidatePipelineShaderStage(pCreateInfo->pStages, pipeline, &module, &entrypoint, false);
Jeff Bolzfbe51582018-09-13 10:01:35 -05002829}
2830
Dave Houltona9df0ce2018-02-07 10:51:23 -07002831uint32_t ValidationCache::MakeShaderHash(VkShaderModuleCreateInfo const *smci) { return XXH32(smci->pCode, smci->codeSize, 0); }
Chris Forbes9a61e082017-07-24 15:35:29 -07002832
Dave Houltona9df0ce2018-02-07 10:51:23 -07002833static ValidationCache *GetValidationCacheInfo(VkShaderModuleCreateInfo const *pCreateInfo) {
John Zulauf25ea2432019-04-05 10:07:38 -06002834 const auto validation_cache_ci = lvl_find_in_chain<VkShaderModuleValidationCacheCreateInfoEXT>(pCreateInfo->pNext);
2835 if (validation_cache_ci) {
John Zulauf146ee802019-04-05 15:31:06 -06002836 return CastFromHandle<ValidationCache *>(validation_cache_ci->validationCache);
Chris Forbes9a61e082017-07-24 15:35:29 -07002837 }
Chris Forbes9a61e082017-07-24 15:35:29 -07002838 return nullptr;
2839}
2840
Mark Lobodzinskib56bbb92019-02-18 11:49:59 -07002841bool CoreChecks::PreCallValidateCreateShaderModule(VkDevice device, const VkShaderModuleCreateInfo *pCreateInfo,
2842 const VkAllocationCallbacks *pAllocator, VkShaderModule *pShaderModule) {
Chris Forbes4ae55b32017-06-09 14:42:56 -07002843 bool skip = false;
2844 spv_result_t spv_valid = SPV_SUCCESS;
Chris Forbes4ae55b32017-06-09 14:42:56 -07002845
Mark Lobodzinskib02a4852019-04-19 12:35:30 -06002846 if (disabled.shader_validation) {
Chris Forbes4ae55b32017-06-09 14:42:56 -07002847 return false;
2848 }
2849
Mark Lobodzinskif45e45f2019-04-19 14:15:39 -06002850 auto have_glsl_shader = device_extensions.vk_nv_glsl_shader;
Chris Forbes4ae55b32017-06-09 14:42:56 -07002851
2852 if (!have_glsl_shader && (pCreateInfo->codeSize % 4)) {
Mark Lobodzinski7767ad82019-03-09 13:35:25 -07002853 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
Dave Houlton78d09922018-05-17 15:48:45 -06002854 "VUID-VkShaderModuleCreateInfo-pCode-01376",
2855 "SPIR-V module not valid: Codesize must be a multiple of 4 but is " PRINTF_SIZE_T_SPECIFIER ".",
2856 pCreateInfo->codeSize);
Chris Forbes4ae55b32017-06-09 14:42:56 -07002857 } else {
Chris Forbes9a61e082017-07-24 15:35:29 -07002858 auto cache = GetValidationCacheInfo(pCreateInfo);
2859 uint32_t hash = 0;
2860 if (cache) {
2861 hash = ValidationCache::MakeShaderHash(pCreateInfo);
Dave Houltona9df0ce2018-02-07 10:51:23 -07002862 if (cache->Contains(hash)) return false;
Chris Forbes9a61e082017-07-24 15:35:29 -07002863 }
2864
Chris Forbes4ae55b32017-06-09 14:42:56 -07002865 // Use SPIRV-Tools validator to try and catch any issues with the module itself
Dave Houlton0ea2d012018-06-21 14:00:26 -06002866 spv_target_env spirv_environment = SPV_ENV_VULKAN_1_0;
Mark Lobodzinski544def72019-04-19 14:25:59 -06002867 if (api_version >= VK_API_VERSION_1_1) {
Dave Houlton0ea2d012018-06-21 14:00:26 -06002868 spirv_environment = SPV_ENV_VULKAN_1_1;
2869 }
2870 spv_context ctx = spvContextCreate(spirv_environment);
Dave Houltona9df0ce2018-02-07 10:51:23 -07002871 spv_const_binary_t binary{pCreateInfo->pCode, pCreateInfo->codeSize / sizeof(uint32_t)};
Chris Forbes4ae55b32017-06-09 14:42:56 -07002872 spv_diagnostic diag = nullptr;
Karl Schultzfda1b382018-08-08 18:56:11 -06002873 spv_validator_options options = spvValidatorOptionsCreate();
Mark Lobodzinskif45e45f2019-04-19 14:15:39 -06002874 if (device_extensions.vk_khr_relaxed_block_layout) {
Karl Schultzfda1b382018-08-08 18:56:11 -06002875 spvValidatorOptionsSetRelaxBlockLayout(options, true);
2876 }
Mark Lobodzinskif45e45f2019-04-19 14:15:39 -06002877 if (device_extensions.vk_ext_scalar_block_layout &&
Mark Lobodzinskid7b03cc2019-04-19 14:23:10 -06002878 enabled_features.scalar_block_layout_features.scalarBlockLayout == VK_TRUE) {
Tobias Hector6a0ece72018-12-10 12:24:05 +00002879 spvValidatorOptionsSetScalarBlockLayout(options, true);
2880 }
Karl Schultzfda1b382018-08-08 18:56:11 -06002881 spv_valid = spvValidateWithOptions(ctx, options, &binary, &diag);
Chris Forbes4ae55b32017-06-09 14:42:56 -07002882 if (spv_valid != SPV_SUCCESS) {
2883 if (!have_glsl_shader || (pCreateInfo->pCode[0] == spv::MagicNumber)) {
Mark Lobodzinski7767ad82019-03-09 13:35:25 -07002884 skip |=
2885 log_msg(report_data, spv_valid == SPV_WARNING ? VK_DEBUG_REPORT_WARNING_BIT_EXT : VK_DEBUG_REPORT_ERROR_BIT_EXT,
2886 VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0, kVUID_Core_Shader_InconsistentSpirv,
2887 "SPIR-V module not valid: %s", diag && diag->error ? diag->error : "(no error text)");
Chris Forbes4ae55b32017-06-09 14:42:56 -07002888 }
Chris Forbes9a61e082017-07-24 15:35:29 -07002889 } else {
2890 if (cache) {
2891 cache->Insert(hash);
2892 }
Chris Forbes4ae55b32017-06-09 14:42:56 -07002893 }
2894
Karl Schultzfda1b382018-08-08 18:56:11 -06002895 spvValidatorOptionsDestroy(options);
Chris Forbes4ae55b32017-06-09 14:42:56 -07002896 spvDiagnosticDestroy(diag);
2897 spvContextDestroy(ctx);
2898 }
2899
Chris Forbes4ae55b32017-06-09 14:42:56 -07002900 return skip;
Mark Lobodzinski01734072019-02-13 17:39:15 -07002901}
2902
Mark Lobodzinskib56bbb92019-02-18 11:49:59 -07002903void CoreChecks::PreCallRecordCreateShaderModule(VkDevice device, const VkShaderModuleCreateInfo *pCreateInfo,
2904 const VkAllocationCallbacks *pAllocator, VkShaderModule *pShaderModule,
2905 void *csm_state_data) {
Mark Lobodzinski1db77e82019-03-01 10:02:54 -07002906 create_shader_module_api_state *csm_state = reinterpret_cast<create_shader_module_api_state *>(csm_state_data);
Mark Lobodzinskib02a4852019-04-19 12:35:30 -06002907 if (enabled.gpu_validation) {
Mark Lobodzinski586d10e2019-03-08 18:19:48 -07002908 GpuPreCallCreateShaderModule(pCreateInfo, pAllocator, pShaderModule, &csm_state->unique_shader_id,
Mark Lobodzinski01734072019-02-13 17:39:15 -07002909 &csm_state->instrumented_create_info, &csm_state->instrumented_pgm);
2910 }
2911}
2912
Mark Lobodzinskib56bbb92019-02-18 11:49:59 -07002913void CoreChecks::PostCallRecordCreateShaderModule(VkDevice device, const VkShaderModuleCreateInfo *pCreateInfo,
2914 const VkAllocationCallbacks *pAllocator, VkShaderModule *pShaderModule,
2915 VkResult result, void *csm_state_data) {
Mark Lobodzinski01734072019-02-13 17:39:15 -07002916 if (VK_SUCCESS != result) return;
Mark Lobodzinski1db77e82019-03-01 10:02:54 -07002917 create_shader_module_api_state *csm_state = reinterpret_cast<create_shader_module_api_state *>(csm_state_data);
Mark Lobodzinski01734072019-02-13 17:39:15 -07002918
Mark Lobodzinski544def72019-04-19 14:25:59 -06002919 spv_target_env spirv_environment = ((api_version >= VK_API_VERSION_1_1) ? SPV_ENV_VULKAN_1_1 : SPV_ENV_VULKAN_1_0);
Mark Lobodzinski01734072019-02-13 17:39:15 -07002920 bool is_spirv = (pCreateInfo->pCode[0] == spv::MagicNumber);
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002921 std::unique_ptr<SHADER_MODULE_STATE> new_shader_module(
2922 is_spirv ? new SHADER_MODULE_STATE(pCreateInfo, *pShaderModule, spirv_environment, csm_state->unique_shader_id)
2923 : new SHADER_MODULE_STATE());
Mark Lobodzinski7767ad82019-03-09 13:35:25 -07002924 shaderModuleMap[*pShaderModule] = std::move(new_shader_module);
Mark Lobodzinski01734072019-02-13 17:39:15 -07002925}
Lockeaa8fdc02019-04-02 11:59:20 -06002926
Mark Lobodzinski3c59d972019-04-25 11:28:14 -06002927bool CoreChecks::ValidateComputeWorkGroupSizes(const SHADER_MODULE_STATE *shader) {
Lockeaa8fdc02019-04-02 11:59:20 -06002928 bool skip = false;
2929 uint32_t local_size_x = 0;
2930 uint32_t local_size_y = 0;
2931 uint32_t local_size_z = 0;
2932 if (FindLocalSize(shader, local_size_x, local_size_y, local_size_z)) {
2933 if (local_size_x > phys_dev_props.limits.maxComputeWorkGroupSize[0]) {
2934 skip |=
2935 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
2936 HandleToUint64(shader->vk_shader_module), "UNASSIGNED-features-limits-maxComputeWorkGroupSize",
Juan A. Suarez Romerof817c0e2019-05-28 12:33:17 +02002937 "ShaderModule %s local_size_x (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[0] (%" PRIu32 ").",
Lockeaa8fdc02019-04-02 11:59:20 -06002938 report_data->FormatHandle(shader->vk_shader_module).c_str(), local_size_x,
2939 phys_dev_props.limits.maxComputeWorkGroupSize[0]);
2940 }
2941 if (local_size_y > phys_dev_props.limits.maxComputeWorkGroupSize[1]) {
2942 skip |=
2943 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
2944 HandleToUint64(shader->vk_shader_module), "UNASSIGNED-features-limits-maxComputeWorkGroupSize",
Juan A. Suarez Romerof817c0e2019-05-28 12:33:17 +02002945 "ShaderModule %s local_size_y (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[1] (%" PRIu32 ").",
Lockeaa8fdc02019-04-02 11:59:20 -06002946 report_data->FormatHandle(shader->vk_shader_module).c_str(), local_size_x,
2947 phys_dev_props.limits.maxComputeWorkGroupSize[1]);
2948 }
2949 if (local_size_z > phys_dev_props.limits.maxComputeWorkGroupSize[2]) {
2950 skip |=
2951 log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
2952 HandleToUint64(shader->vk_shader_module), "UNASSIGNED-features-limits-maxComputeWorkGroupSize",
Juan A. Suarez Romerof817c0e2019-05-28 12:33:17 +02002953 "ShaderModule %s local_size_z (%" PRIu32 ") exceeds device limit maxComputeWorkGroupSize[2] (%" PRIu32 ").",
Lockeaa8fdc02019-04-02 11:59:20 -06002954 report_data->FormatHandle(shader->vk_shader_module).c_str(), local_size_x,
2955 phys_dev_props.limits.maxComputeWorkGroupSize[2]);
2956 }
2957
2958 uint32_t limit = phys_dev_props.limits.maxComputeWorkGroupInvocations;
2959 uint64_t invocations = local_size_x * local_size_y;
2960 // Prevent overflow.
2961 bool fail = false;
2962 if (invocations > UINT32_MAX || invocations > limit) {
2963 fail = true;
2964 }
2965 if (!fail) {
2966 invocations *= local_size_z;
2967 if (invocations > UINT32_MAX || invocations > limit) {
2968 fail = true;
2969 }
2970 }
2971 if (fail) {
2972 skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SHADER_MODULE_EXT,
2973 HandleToUint64(shader->vk_shader_module), "UNASSIGNED-features-limits-maxComputeWorkGroupInvocations",
Juan A. Suarez Romerof817c0e2019-05-28 12:33:17 +02002974 "ShaderModule %s local_size (%" PRIu32 ", %" PRIu32 ", %" PRIu32
Lockeaa8fdc02019-04-02 11:59:20 -06002975 ") exceeds device limit maxComputeWorkGroupInvocations (%" PRIu32 ").",
2976 report_data->FormatHandle(shader->vk_shader_module).c_str(), local_size_x, local_size_y, local_size_z,
2977 limit);
2978 }
2979 }
2980 return skip;
2981}