[gfx] Inline raytracing fixes in response to code review. (shader-slang#1905)

Co-authored-by: Yong He <yhe@nvidia.com>

Author: csyonghe

examples/ray-tracing/main.cpp

@@ -324,7 +324,7 @@ Slang::Result initialize()
 
     IBufferResource::Desc transformBufferDesc;
     transformBufferDesc.type = IResource::Type::Buffer;
-    transformBufferDesc.sizeInBytes = sizeof(float) * 16;
+    transformBufferDesc.sizeInBytes = sizeof(float) * 12;
     transformBufferDesc.defaultState = ResourceState::ShaderResource;
     float transformData[12] = {
         1.0f, 0.0f, 0.0f, 0.0f, 0.0f, 1.0f, 0.0f, 0.0f, 0.0f, 0.0f, 1.0f, 0.0f};

slang-gfx.h

@@ -520,6 +520,8 @@ class IAccelerationStructure : public IResourceView
 
     struct BuildFlags
     {
+        // The enum values are intentionally consistent with
+        // D3D12_RAYTRACING_ACCELERATION_STRUCTURE_BUILD_FLAGS.
         enum Enum
         {
             None,
@@ -539,6 +541,8 @@ class IAccelerationStructure : public IResourceView
 
     struct GeometryFlags
     {
+        // The enum values are intentionally consistent with
+        // D3D12_RAYTRACING_GEOMETRY_FLAGS.
         enum Enum
         {
             None,
@@ -571,7 +575,7 @@ class IAccelerationStructure : public IResourceView
 
     struct ProceduralAABBDesc
     {
-        // Number of AABBs.
+        /// Number of AABBs.
         uint64_t count;
 
         /// Pointer to an array of `ProceduralAABB` values in device memory.
@@ -594,6 +598,8 @@ class IAccelerationStructure : public IResourceView
 
     struct GeometryInstanceFlags
     {
+        // The enum values are kept consistent with D3D12_RAYTRACING_INSTANCE_FLAGS
+        // and VkGeometryInstanceFlagBitsKHR.
         enum Enum : uint32_t
         {
             None = 0,
@@ -604,6 +610,8 @@ class IAccelerationStructure : public IResourceView
         };
     };
 
+    // The layout of this struct is intentionally consistent with D3D12_RAYTRACING_INSTANCE_DESC
+    // and VkAccelerationStructureInstanceKHR.
     struct InstanceDesc
     {
         float transform[3][4];

tools/gfx/d3d12/render-d3d12.cpp

@@ -405,7 +405,6 @@ class D3D12Device : public RendererBase
         }
 
     public:
-        uint32_t m_stride = 0;
         Result init(const IQueryPool::Desc& desc, D3D12Device* device, uint32_t stride);
 
         virtual SLANG_NO_THROW Result SLANG_MCALL
@@ -428,6 +427,7 @@ class D3D12Device : public RendererBase
         QueryType m_queryType;
         RefPtr<BufferResourceImpl> m_bufferResource;
         RefPtr<D3D12Device> m_device;
+        uint32_t m_stride = 0;
     };
 
     struct BoundVertexBuffer

tools/render-test/render-test-main.cpp

@@ -681,7 +681,7 @@ void RenderTestApp::_initializeAccelerationStructure()
 
     IBufferResource::Desc transformBufferDesc;
     transformBufferDesc.type = IResource::Type::Buffer;
-    transformBufferDesc.sizeInBytes = sizeof(float) * 16;
+    transformBufferDesc.sizeInBytes = sizeof(float) * 12;
     transformBufferDesc.defaultState = ResourceState::ShaderResource;
     float transformData[12] = {
         1.0f, 0.0f, 0.0f, 0.0f, 0.0f, 1.0f, 0.0f, 0.0f, 0.0f, 0.0f, 1.0f, 0.0f};