shader-slang / slang-rhi / 19232492817

23%

push

github

Unify functions for loading test shaders (#569)

* Unify shader loading functions into single loadProgram API

Replaced loadComputeProgram, loadGraphicsProgram, and loadShaderPrograms
with a single loadProgram function that:
- Accepts multiple entry points
- Makes reflection optional (nullptr skips linking for ray tracing)
- Accepts optional slang::ISession for precompiled modules

NOTE: We use a std::vector instead of a span for the `entryPointNames`
parameter so we can use braced initializers when specifying shader names.

Updated all call sites to use cleaner syntax:
  loadProgram(device, program, "module", {"computeMain"}, &reflection);

* Refactor shader loading API to separate linking from loading

Split loadProgram into two distinct functions:
- loadProgram: loads shader programs without linking (no reflection)
- loadAndLinkProgram: loads and links programs with reflection support

Both functions now have overloads that omit the slangSession parameter,
defaulting to nullptr for simpler call sites. Parameter order updated
to place outShaderProgram last for consistency.

Updated all test files to:
- Use loadProgram for tests without reflection needs
- Use loadAndLinkProgram for tests requiring reflection data
- Remove unused slangReflection variables
- Use simpler overloads without explicit session parameter

* clang-format

* Delete ray-tracing-common.cpp and inline launchPipeline

launchPipeline was the only function in ray-tracing-common.cpp. Inline
the function into ray-tracing-common.h and delete the basically empty
cpp file.

* Add single entry point overloads and document new functions

* fix missed single entry point call

* fix a few more single entry point calls

---------

Co-authored-by: Simon Kallweit <64953474+skallweitNV@users.noreply.github.com>

4124 of 17036 branches covered (24.21%)

Branch coverage included in aggregate %.

14392 of 41517 relevant lines covered (34.67%)

86.69 hits per line