add missing Result to IRayTracingCommandEncoder::bindPipline (shader-slang#4148)

Author: skallweitNV

slang-gfx.h

@@ -1842,7 +1842,7 @@ class IRayTracingCommandEncoder : public IResourceCommandEncoder
     virtual SLANG_NO_THROW void SLANG_MCALL
         deserializeAccelerationStructure(IAccelerationStructure* dest, DeviceAddress source) = 0;
 
-    virtual SLANG_NO_THROW void SLANG_MCALL
+    virtual SLANG_NO_THROW Result SLANG_MCALL
         bindPipeline(IPipelineState* state, IShaderObject** outRootObject) = 0;
     // Sets the current pipeline state along with a pre-created mutable root shader object.
     virtual SLANG_NO_THROW Result SLANG_MCALL

tools/gfx/d3d12/d3d12-command-encoder.cpp

@@ -1339,10 +1339,10 @@ void RayTracingCommandEncoderImpl::deserializeAccelerationStructure(
         D3D12_RAYTRACING_ACCELERATION_STRUCTURE_COPY_MODE_DESERIALIZE);
 }
 
-void RayTracingCommandEncoderImpl::bindPipeline(
+Result RayTracingCommandEncoderImpl::bindPipeline(
     IPipelineState* state, IShaderObject** outRootObject)
 {
-    bindPipelineImpl(state, outRootObject);
+    return bindPipelineImpl(state, outRootObject);
 }
 
 Result RayTracingCommandEncoderImpl::dispatchRays(

tools/gfx/d3d12/d3d12-command-encoder.h

@@ -328,7 +328,7 @@ class RayTracingCommandEncoderImpl
         serializeAccelerationStructure(DeviceAddress dest, IAccelerationStructure* source) override;
     virtual SLANG_NO_THROW void SLANG_MCALL deserializeAccelerationStructure(
         IAccelerationStructure* dest, DeviceAddress source) override;
-    virtual SLANG_NO_THROW void SLANG_MCALL
+    virtual SLANG_NO_THROW Result SLANG_MCALL
         bindPipeline(IPipelineState* state, IShaderObject** outRootObject) override;
     virtual SLANG_NO_THROW Result SLANG_MCALL
         bindPipelineWithRootObject(IPipelineState* state, IShaderObject* rootObject) override

tools/gfx/debug-layer/debug-command-encoder.cpp

@@ -470,16 +470,17 @@ void DebugRayTracingCommandEncoder::deserializeAccelerationStructure(
     baseObject->deserializeAccelerationStructure(getInnerObj(dest), source);
 }
 
-void DebugRayTracingCommandEncoder::bindPipeline(
+Result DebugRayTracingCommandEncoder::bindPipeline(
     IPipelineState* state, IShaderObject** outRootObject)
 {
     SLANG_GFX_API_FUNC;
     auto innerPipeline = getInnerObj(state);
     IShaderObject* innerRootObject = nullptr;
     commandBuffer->rootObject.reset();
-    baseObject->bindPipeline(innerPipeline, &innerRootObject);
+    Result result = baseObject->bindPipeline(innerPipeline, &innerRootObject);
     commandBuffer->rootObject.baseObject.attach(innerRootObject);
     *outRootObject = &commandBuffer->rootObject;
+    return result;
 }
 
 Result DebugRayTracingCommandEncoder::bindPipelineWithRootObject(

tools/gfx/debug-layer/debug-command-encoder.h

@@ -276,7 +276,7 @@ class DebugRayTracingCommandEncoder
     virtual SLANG_NO_THROW void SLANG_MCALL deserializeAccelerationStructure(
         IAccelerationStructure* dest,
         DeviceAddress source) override;
-    virtual SLANG_NO_THROW void SLANG_MCALL
+    virtual SLANG_NO_THROW Result SLANG_MCALL
         bindPipeline(IPipelineState* state, IShaderObject** outRootObject) override;
     virtual SLANG_NO_THROW Result SLANG_MCALL
         bindPipelineWithRootObject(IPipelineState* state, IShaderObject* rootObject) override;

tools/gfx/gfx.slang

@@ -1519,7 +1519,7 @@ public interface IRayTracingCommandEncoder : IResourceCommandEncoder
     public void serializeAccelerationStructure(DeviceAddress dest, IAccelerationStructure source);
     public void deserializeAccelerationStructure(IAccelerationStructure dest, DeviceAddress source);
 
-    public void bindPipeline(IPipelineState state, out IShaderObject rootObject);
+    public Result bindPipeline(IPipelineState state, out IShaderObject rootObject);
     // Sets the current pipeline state along with a pre-created mutable root shader object.
     public Result bindPipelineWithRootObject(IPipelineState state, IShaderObject rootObject);
 

tools/gfx/vulkan/vk-command-encoder.cpp

@@ -1443,9 +1443,9 @@ void RayTracingCommandEncoder::deserializeAccelerationStructure(
         m_commandBuffer->m_commandBuffer, &copyInfo);
 }
 
-void RayTracingCommandEncoder::bindPipeline(IPipelineState* pipeline, IShaderObject** outRootObject)
+Result RayTracingCommandEncoder::bindPipeline(IPipelineState* pipeline, IShaderObject** outRootObject)
 {
-    setPipelineStateImpl(pipeline, outRootObject);
+    return setPipelineStateImpl(pipeline, outRootObject);
 }
 
 Result RayTracingCommandEncoder::bindPipelineWithRootObject(

tools/gfx/vulkan/vk-command-encoder.h

@@ -336,7 +336,7 @@ class RayTracingCommandEncoder
     virtual SLANG_NO_THROW void SLANG_MCALL deserializeAccelerationStructure(
         IAccelerationStructure* dest, DeviceAddress source) override;
 
-    virtual SLANG_NO_THROW void SLANG_MCALL
+    virtual SLANG_NO_THROW Result SLANG_MCALL
         bindPipeline(IPipelineState* pipeline, IShaderObject** outRootObject) override;
 
     virtual SLANG_NO_THROW Result SLANG_MCALL bindPipelineWithRootObject(