Making it easier to work with shaders https://git.yummers.dev/slang.git/atom?h=master 2024-10-29T06:49:26+00:00 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 2023-02-17T00:44:04+00:00 Yong He yonghe@outlook.com 2023-02-17T00:44:04+00:00 urn:sha1:245466d89cfe54b78da486f06d470bc6daaf4625 Co-authored-by: Yong He <yhe@nvidia.com> 2021-11-10T18:48:44+00:00 Theresa Foley tfoleyNV@users.noreply.github.com 2021-11-10T18:48:44+00:00 urn:sha1:95e82acc0b32c81a9c6ac39708d18a423d8c7b1e Recent work that added support for translating DXR-style ray tracing shaders to work with OptiX seems to have accidentally applied its transformations even when compute shaders are translated for CUDA. As a result, compute entry points with `uniform` parameters at entry-point scope would be miscompiled to use OptiX calls that are not available for non-OptiX compiles. This change fixes the relevant pass so that it correctly opts-out on compute entry points, and also unifies some pieces of code that were being shared between a few different IR passes but that had gotten copy-pasted for the OptiX case. The fix has been confirmed by running relevant CUDA tests locally, but CUDA is still disabled in the default CI builds, so this change is not yet actively being tested to avoid further regression.