Update slang-rhi and fix error handling (shader-slang#5485)

Update slang-rhi to pick up a fix in webgpu compilation error handling.

In doRenderComparisonTestRun(), only return TestResult::Pass if the
shader is ran actually compiled.

A similar check is in place elsewhere in slang-test-main, but was
missed in doRenderComparisonTestRun().

Add two tests to the github CI skiplist, and use the skiplist in one
additional CI config that was running without it.

Closes 5291

Author: cheneym2

.github/workflows/ci.yml

@@ -145,7 +145,8 @@ jobs:
               -use-test-server \
               -server-count 8 \
               -category ${{ matrix.test-category }} \
-              -api all-cpu
+              -api all-cpu \
+              -expected-failure-list tests/expected-failure-github.txt
           elif [[ "${{matrix.has-gpu}}" == "true" ]]; then
             "$bin_dir/slang-test" \
               -use-test-server \

external/slang-rhi

@@ -17,6 +17,7 @@ tests/autodiff/global-param-hoisting.slang.4 syn (wgpu)
 tests/autodiff/material/diff-bwd-falcor-material-system.slang.2 syn (wgpu)
 tests/autodiff/material2/diff-bwd-falcor-material-system.slang.2 syn (wgpu)
 tests/autodiff/matrix-arithmetic-fwd.slang.2 syn (wgpu)
+tests/autodiff/no-diff-strip.slang.3 syn (wgpu)
 tests/autodiff/reverse-loop-checkpoint-test.slang.3 syn (wgpu)
 tests/bindings/nested-parameter-block-2.slang.4 syn (wgpu)
 tests/bindings/nested-parameter-block-3.slang.4 syn (wgpu)
@@ -39,6 +40,7 @@ tests/bugs/op-assignment-unify-mat.slang.4 syn (wgpu)
 tests/bugs/shadowed-lookup.slang.1 syn (wgpu)
 tests/bugs/specialize-function-array-args.slang.2 syn (wgpu)
 tests/bugs/static-var.slang.1 syn (wgpu)
+tests/bugs/texture2d-gather.hlsl.2 syn (wgpu)
 tests/bugs/user-attribute-lookup.slang.2 syn (wgpu)
 tests/compute/atomics (wgpu)
 tests/compute/atomics-buffer (wgpu)

tests/expected-failure-github.txt

@@ -3583,6 +3583,11 @@ TestResult doRenderComparisonTestRun(
 
     *outOutput = output;
 
+    // Always fail if the compilation produced a failure.
+    if (exeRes.resultCode != 0)
+    {
+        return TestResult::Fail;
+    }
     return TestResult::Pass;
 }