Making it easier to work with shaders https://git.yummers.dev/slang.git/atom?h=master 2025-10-06T20:55:39+00:00 2025-10-06T20:55:39+00:00 Sami Kiminki (NVIDIA) 235843927+skiminki-nv@users.noreply.github.com 2025-10-06T20:55:39+00:00 urn:sha1:f4449d9376f0a5c32274fd812d135f3595159bae Test `slang-unit-test-tool/cudaCodeGenBug.internal` requires that the CUDA toolkit is available. Add a check for the NVRTC backend to avoid a failure when this is not the case. Fixes #6636 2025-05-02T23:48:50+00:00 Jay Kwak 82421531+jkwak-work@users.noreply.github.com 2025-05-02T23:48:50+00:00 urn:sha1:7f9283a34b4aaf3401cdb652a2f9208b2b4ff4f4 `cudaCodeGenBug` is expected to fail on Linux, because the variable `code` is nullptr. When the next test tried to dereference, it causes a seg-fault. 2025-03-05T06:16:29+00:00 Anders Leino aleino@nvidia.com 2025-03-05T06:16:29+00:00 urn:sha1:5248a0254a48382d06ecb190c9f87c0ab62ff534 * Add cuda codegen bug repro This just compiles tests/compute/simlpe.slang for PTX with the new compilation API, in order to reproduce a code generation bug. * Detect entrypoint more robustly when applying ConstRef hack during lowring For shaders like tests/compute/simple.slang, which have a 'numthreads' attribute but no 'shader' attribute, the old compile request API would add an EntryPointAttribute to the AST node of the entry point. However, the new API doesn't, and so a certain ConstRef hack doesn't get applied when using the new API, leading to subsequent code generation issues. This patch also checks for a 'numthreads' attribute when deciding whether to apply the ConstRef hack. This closes issue #6507 and helps to resolve issue #4760. * Add expected failure list for GitHub runners Our GitHub runners don't have the CUDA toolkits installed, so they can't run all tests. 2024-11-14T04:34:18+00:00 Ellie Hermaszewska ellieh@nvidia.com 2024-11-14T04:34:18+00:00 urn:sha1:7b570feed42976a6e787d79a70aaf8e667745e58 This stops adding the repo root to the include path for anything linking with slang. This enabled a bunch of convenient includes, but might lead to confusing behavior for anyone including slang. Not to mention differences including it from an install vs source. Co-authored-by: Yong He <yonghe@outlook.com> 2024-10-29T06:49:26+00:00 Ellie Hermaszewska ellieh@nvidia.com 2024-10-29T06:49:26+00:00 urn:sha1:f65d756bff8d4c5cbc15bd0322a2ae8e6b896a21 * format * Minor test fixes * enable checking cpp format in ci 2024-10-09T07:39:38+00:00 Yong He yonghe@outlook.com 2024-10-09T07:39:38+00:00 urn:sha1:bea1394ad35680940a0b69b9c67efc43764cc194 2024-07-17T17:53:19+00:00 kaizhangNV 149626564+kaizhangNV@users.noreply.github.com 2024-07-17T17:53:19+00:00 urn:sha1:2db15080085856ed9b5f20642dbb354aac59a888 * Move the file public header files to `include` dir Close the issue (#4635). Move the following headers files to a `include` dir located at root dir of slang repo: slang-com-helper.h -> include/slang-com-helper.h slang-com-ptr.h -> include/slang-com-ptr.h slang-gfx.h -> include/slang-gfx.h slang.h -> include/slang.h Change cmake/SlangTarget.cmake to add include path to every target, and change the source file to use "#include <slang.h>" to include the public headers. The source code update is by the script like follow: ``` fileNames_slang=$(grep -r "\".*slang\.h\"" source/ -l) for fileName in "${fileNames_slang[@]}" do echo "$fileName" sed -i "s/\".*slang\.h\"/\"slang\.h\"/" $fileName done ``` * Fix the test issues * Fix cpu test issues by adding include seach path * Update cmake to not add include path for every target Also change "#include <slang.h>" to "include "slang.h" " to make the coding style consistent with other slang code. * Change public include to private include for unit-test and slang-glslang 2024-04-09T19:47:03+00:00 Yong He yonghe@outlook.com 2024-04-09T19:47:03+00:00 urn:sha1:6a465a4db65b924b03930261da3b64b1c792ef85 * Allow COM based API to discover and check entrypoints without [shader] attribute. * Undo changes. * More comments.