commit 148d94edaa92be417d5417b0f263edda46cba28c
author Jeff Bolz <jbolz@nvidia.com> Thu Dec 13 21:25:56 2018 -0600
committer Tobin Ehlis <tobine@google.com> Wed Jan 02 13:07:24 2019 -0700
tree e6ffef71ee16894a7cd1f86113c42a4d110f5686
parent 62c5dba8da33a265f6fbd4830496a5ca12731efd

layers: raytracing and descriptor indexing fixes

https://github.com/KhronosGroup/SPIRV-Tools/pull/2190 fixes a crash caused
by the OpTypeAccelerationStructureNV instruction being deleted. Also add
some missing shader stages for raytracing and mesh shaders.

layers/shader_validation.cpp

diff --git a/layers/shader_validation.cpp b/layers/shader_validation.cpp
index c9c2024..b6e78dc 100644
--- a/layers/shader_validation.cpp
+++ b/layers/shader_validation.cpp
@@ -1567,8 +1567,16 @@
     if (has_writable_descriptor) {
         switch (stage) {
             case VK_SHADER_STAGE_COMPUTE_BIT:
+            case VK_SHADER_STAGE_RAYGEN_BIT_NV:
+            case VK_SHADER_STAGE_ANY_HIT_BIT_NV:
+            case VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV:
+            case VK_SHADER_STAGE_MISS_BIT_NV:
+            case VK_SHADER_STAGE_INTERSECTION_BIT_NV:
+            case VK_SHADER_STAGE_CALLABLE_BIT_NV:
+            case VK_SHADER_STAGE_TASK_BIT_NV:
+            case VK_SHADER_STAGE_MESH_BIT_NV:
                 /* No feature requirements for writes and atomics from compute
-                 * stage */
+                 * raytracing, or mesh stages */
                 break;
             case VK_SHADER_STAGE_FRAGMENT_BIT:
                 skip |= RequireFeature(report_data, features->core.fragmentStoresAndAtomics, "fragmentStoresAndAtomics");
@@ -1793,6 +1801,16 @@
             }
             break;
 
+        case VK_SHADER_STAGE_RAYGEN_BIT_NV:
+        case VK_SHADER_STAGE_ANY_HIT_BIT_NV:
+        case VK_SHADER_STAGE_CLOSEST_HIT_BIT_NV:
+        case VK_SHADER_STAGE_MISS_BIT_NV:
+        case VK_SHADER_STAGE_INTERSECTION_BIT_NV:
+        case VK_SHADER_STAGE_CALLABLE_BIT_NV:
+        case VK_SHADER_STAGE_TASK_BIT_NV:
+        case VK_SHADER_STAGE_MESH_BIT_NV:
+            break;
+
         default:
             assert(false);  // This should never happen
     }