Making it easier to work with shaders https://git.yummers.dev/slang.git/atom?h=master 2025-10-16T22:16:18+00:00 2025-10-16T22:16:18+00:00 aidanfnv aidanf@nvidia.com 2025-10-16T22:16:18+00:00 urn:sha1:a890ddece9ab69b549068605595d006481a59d0f 2025-10-03T04:48:11+00:00 Theresa Foley 10618364+tangent-vector@users.noreply.github.com 2025-10-03T04:48:11+00:00 urn:sha1:cc8f6a241edb47c43c5698ee33abed4fe57d4566 Note that while this change touched a large numer of files, there are no changes to functionality being made here. The only things being done are renaming various symbols and, in a few cases, updating or adding comments for consistency with the new names. The core of the naming changes are: * Most things named to refer to `OutType` (e.g., `IROutType`, `IRBuilder::getOutType()`, etc.) have been consistently renamed to refer to `OutParamType`, to emphasize that the relevant AST/IR node types are only intended for use to represent `out` parameters. * The same change as described above for `OutType` is also made for `RefType`, which becomes `RefParamType` in most cases. One mess that this exposes is the way that the `ExplicitRef<T>` type in the core module currently lowers to `IRRefParamType`. This change sticks to the rule of not making functional changes, so that mess is left as-is for now. * Names referring to `InOutType` have been changed to instead refer to `BorrowInOutType`. The intention with this naming change is to emphasize that the Slang rules for `inout` are semantically those of a borrow (or at least our interpretation of what a borrow means). * Names referring to `ConstRefType` have been changed to instead refer to `BorrowInType`. This change starts work on clarifying that the existing `__constref` modifier was never intended to be a read-only analogue of `__ref`, and instead is the input-only analogue of `inout`. * The `ParameterDirection` enum type has been changed to `ParamPassingMode`, to reflect the fact that the concept of "direction" fails to capture what is actually being encoded, particularly once we have modes beyond simple `in`/`out`/`inout`. While this change does not alter behavior in any case (the user-exposed Slang language is unchanged), it is intended to set up subsequence changes that will work to make the handling of these types in the compiler more nuanced and correct. Breaking this part of the change out separately is primarily motivated by a desire to minimize the effort for reviewers. --------- Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com> 2025-10-01T16:17:04+00:00 Harsh Aggarwal (NVIDIA) haaggarwal@nvidia.com 2025-10-01T16:17:04+00:00 urn:sha1:4d241f2ccc2b07673615e94ea8e99060cf2da66c 2025-09-25T04:49:26+00:00 Jay Kwak 82421531+jkwak-work@users.noreply.github.com 2025-09-25T04:49:26+00:00 urn:sha1:db44c1b732891102199f0ec9d219a33d2ab2d6a1 Related to - https://github.com/shader-slang/slang/issues/8519 2025-09-10T02:34:44+00:00 Jay Kwak 82421531+jkwak-work@users.noreply.github.com 2025-09-10T02:34:44+00:00 urn:sha1:7026dec790ab0c239c58ebf72b17633e9e8765c3 This updates SPIRV submodules. Unfortunately we cannot use the latest because one of PRs in SPIRV-Tools causes VVL errors on our slang-tests. The issue is tracked in - https://github.com/KhronosGroup/SPIRV-Tools/issues/6316 - https://github.com/shader-slang/slang/issues/8414 This PR also updates/simplifies the document of updating SPIRV process. 2025-06-30T23:43:16+00:00 kaizhangNV 149626564+kaizhangNV@users.noreply.github.com 2025-06-30T23:43:16+00:00 urn:sha1:ca21304d0bca7fdde9f5ab3f23129a553dd2408b Close #7491. As the PR KhronosGroup/SPIRV-Tools#6198 is already merged, we can switch spirv-tools repo to upstream repo now. 2025-06-27T00:13:27+00:00 kaizhangNV 149626564+kaizhangNV@users.noreply.github.com 2025-06-27T00:13:27+00:00 urn:sha1:b10319c1c513de8dd7a1d7cf5a45265589978654 * Update spirv-tool submodule to include the debug instruct fix Update spirv-tool to our forked repo that include the fix of spirv-val bug. It mistakenly relocates the `DebugFunctionDefinition`. The upstream PR is: https://github.com/KhronosGroup/SPIRV-Tools/pull/6198 But it's still pending on review. Issue #7491 will keep track the process, when the upstream PR is merged, we need to restore the repo back to upstream repo. 2025-06-26T08:22:30+00:00 Gangzheng Tong tonggangzheng@gmail.com 2025-06-26T08:22:30+00:00 urn:sha1:3a54d68a6ab0a0beb92e9d61befd7280d474531f 2025-06-21T02:07:33+00:00 jarcherNV jarcher@nvidia.com 2025-06-21T02:07:33+00:00 urn:sha1:5510278792ccd1bc7b5bd2053e06c7d2d6d97798 Update spirv-tools to commit: 108b19e5c6979f496deffad4acbe354237afa7d3 Update spirv-headers to commit: 2a611a970fdbc41ac2e3e328802aed9985352dca Update spirv-tools-generated from the spirv-tools build. 2025-06-05T20:03:34+00:00 jarcherNV jarcher@nvidia.com 2025-06-05T20:03:34+00:00 urn:sha1:9b6e9de8f188222788fed0b1a36522ed33ab7f14 Update spirv-tools to take the DebugBuildIdentifier changes. These are needed for upcoming separate debug spirv output. Additionally update spirv-headers to fd966619.