Making it easier to work with shaders https://git.yummers.dev/slang.git/atom?h=master 2025-08-28T17:45:07+00:00 2025-08-28T17:45:07+00:00 Jay Kwak 82421531+jkwak-work@users.noreply.github.com 2025-08-28T17:45:07+00:00 urn:sha1:c19e2e92ae8e713225262a17f39a438cd511d416 When `SIMPLE` type test is used with `-g[1-3]` option, the filecheck pattern will most likely to match to the string itself on the embedded source code rather than match to the emitted spirv-asm code. This commit avoids the problem by removing the embedded source code. This commit also provides an option to keep the embedded source code, `-preserve-embedded-source`. The source code removal is happening in two steps: 1. iterate all output lines and find SPIRV-ASM in the following pattern: `%N = OpExtInst %void %M DebugSource %fileId %sourceId`. And then, store the "%sourceId" value to identify which SPIRV instructions are for the embedded source code. 2. iterate all output lines again to find the `%sourceId = OpString "...."` and replace the whole string with the following string, ``` %1 = OpString "// slang-test removed the embedded source // Use `-preserve-embedded-source` to keep it explicitly " ``` This change revealed problems in the existing tests: - tests/bugs/spirv-debug-info.slang : The expected text was missing and it had to be added. The file also had Carrage-Return character on all lines and the pre-commit git hook removed them. - tests/spirv/debug-info.slang : the expected keyword DebugValue had to change to DebugDeclare, because that's what we get with ToT. - tests/spirv/debug-value-dynamic-index.slang : This test is currently failing, and it will pass once DebugLocalVariable instruction missing for parameter of the entry point function #7693 is resolved. --------- Co-authored-by: slangbot <ellieh+slangbot@nvidia.com> 2023-08-16T03:55:21+00:00 Yong He yonghe@outlook.com 2023-08-16T03:55:21+00:00 urn:sha1:c16f711d83df90f6826a65d61fc56cdbb932c92c 2023-08-15T00:58:29+00:00 Yong He yonghe@outlook.com 2023-08-15T00:58:29+00:00 urn:sha1:b05b126e0975f84e3505b2271e06d567e1c13692 * Update target-deps.json Update glslang binaries. * Fix. --------- Co-authored-by: Yong He <yhe@nvidia.com> 2023-08-11T20:11:15+00:00 Yong He yonghe@outlook.com 2023-08-11T20:11:15+00:00 urn:sha1:e689d5ee8e9724fee018aa14be24f9679ec5c851 * Make sure glsl source in spirv is included when compiling with -g3. * Exclude vulkan tests on github linux CI. * Fix. --------- Co-authored-by: Yong He <yhe@nvidia.com>