Fix spirv legalization of nested ararys. (#3567)

* Fix spirv legalization of nested ararys. * Fix test.
shader-slang · Feb 11, 2024 · 03cddba · 03cddba
1 parent 20ab161
commit 03cddba
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/source/slang/slang-ir-spirv-legalize.cpp b/source/slang/slang-ir-spirv-legalize.cpp
@@ -258,8 +258,7 @@ struct SPIRVLegalizationContext : public SourceEmitterBase
         }
 
         for (auto i : instsToRemove)
-            if (!i->hasUses())
-                i->removeAndDeallocate();
+            i->removeAndDeallocate();
     }
 
     // Returns true if the given type that should be decorated as in `UniformConstant` address space.

diff --git a/tests/spirv/array-of-array.slang b/tests/spirv/array-of-array.slang
@@ -0,0 +1,12 @@
+//TEST:SIMPLE(filecheck=CHECK):-target spirv -entry main -stage compute -emit-spirv-directly -skip-spirv-validation
+
+Texture2D texArray[3][4][5];
+RWStructuredBuffer<float4> outputBuffer;
+
+// CHECK: OpEntryPoint
+
+[shader("compute")]
+void main(uint3 tid : SV_DispatchThreadID)
+{
+    outputBuffer[0] = texArray[tid.x][tid.y][tid.z].Load(int3(1,2,3));
+}