From cd9e1f67184c1361558e18993e5cb392dc1131f0 Mon Sep 17 00:00:00 2001
From: Lujin Wang <143145775+lujinwangnv@users.noreply.github.com>
Date: Fri, 22 Aug 2025 13:48:20 -0700
Subject: Fix mesh shader OutputIndices subscript error by adding missing ref
 accessor (#7929)

Fixes the Slang compiler internal error "subscript had no getter" when
reading from mesh shader output index arrays (e.g., `triangles[0].x`).

## Problem
The `OutputIndices` struct was missing a `ref` accessor in its
`__subscript` implementation, causing the compiler to fail when trying
to materialize subscript expressions as r-values.

## Solution
Added the missing `ref` accessor to `OutputIndices.__subscript` using
the `kIROp_MeshOutputRef` intrinsic operation, matching the pattern used
in `OutputVertices` and `OutputPrimitives`.

## Files Changed
- `source/slang/core.meta.slang` - Added missing `ref` accessor
- `tests/bugs/gh-7925.slang` - Test case to reproduce and verify the fix

Fixes #7925

Generated with [Claude Code](https://claude.ai/code)

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Lujin Wang <lujinwangnv@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: slangbot <ellieh+slangbot@nvidia.com>
Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com>
---
 tests/bugs/gh-7925.slang | 54 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 54 insertions(+)
 create mode 100644 tests/bugs/gh-7925.slang

(limited to 'tests/bugs')

diff --git a/tests/bugs/gh-7925.slang b/tests/bugs/gh-7925.slang
new file mode 100644
index 000000000..470a039c3
--- /dev/null
+++ b/tests/bugs/gh-7925.slang
@@ -0,0 +1,54 @@
+//TEST:SIMPLE(filecheck=CHECK): -target spirv -profile sm_6_6 -stage mesh -entry main
+
+struct MeshOutput
+{
+    float4 position : SV_Position;
+    float3 color : COLOR0;
+}
+
+// Uniform variable for time-based animation
+uniform uint iTime;
+
+[shader("mesh")]
+[outputtopology("triangle")]
+[numthreads(1, 1, 1)]
+void main(
+    in uint tig : SV_GroupIndex,
+    OutputVertices<MeshOutput, 3> vertices,
+    OutputIndices<uint3, 1> triangles
+)
+{
+    // Set the number of vertices and primitives
+    SetMeshOutputCounts(3, 1);
+
+    // Define the triangle indices
+    triangles[0] = uint3(0, 1, 2);
+
+    // Use uiColor as in the original shader
+    // This line should now work after the fix - it was causing the "subscript had no getter" error
+    uint uiColor = triangles[0].x + iTime;
+
+    // Create triangle vertices with time-based animation
+    float timeOffset = float(iTime) * 0.01; // Convert to float and scale
+    float colorOffset = float(uiColor) * 0.005; // Use uiColor for color variation
+
+    // Vertex 0 - Red vertex (top) with uiColor influence
+    vertices[0].position = float4(0.0, 0.5 + sin(timeOffset) * 0.1, 0.0, 1.0);
+    vertices[0].color = float3(1.0, (sin(colorOffset) + 1.0) * 0.5, 0.0); // Red with green modulation from uiColor
+
+    // Vertex 1 - Green vertex (bottom left)
+    vertices[1].position = float4(-0.5, -0.5, 0.0, 1.0);
+    vertices[1].color = float3(0.0, 1.0, 0.0); // Green
+
+    // Vertex 2 - Blue vertex (bottom right)
+    vertices[2].position = float4(0.5, -0.5, 0.0, 1.0);
+    vertices[2].color = float3(0.0, 0.0, 1.0); // Blue
+}
+
+// CHECK: OpCapability MeshShadingEXT
+// CHECK: OpExtension "SPV_EXT_mesh_shader"
+// CHECK: OpEntryPoint MeshEXT %main "main"
+// CHECK: OpExecutionMode %main OutputPrimitivesEXT 1
+// CHECK: OpExecutionMode %main OutputVertices 3
+// CHECK: OpExecutionMode %main LocalSize 1 1 1
+// CHECK: OpExecutionMode %main OutputTrianglesEXT
-- 
cgit v1.2.3