shader-slang / slang-rhi / 14525294141 / 2

23%

main: 23%

push

github

metal: parameter block fix (#303)

* metal: parameter block fix

- When set element type layout for Metal target, if the bindingType is
  parameter block, we will add the container type, `ParameterBlock`, to
  indicate that.

- When creating ShaderObject on Metal target, we have to use
  'MetalArgumentBufferTier2' when the type layout is parameter block to
  get the uniformSize, otherwise, it will always be 0.

- Fix the issue in calculating the offset of argument buffer. We should
  use "MetalArgumentBufferTier2" consistently.

- Simplify the ordinary data writing. We can write the orinary data at
  beginning, the offsets where resource types located will just be 0.
  And following up process will fill up those offsets with device
  address (for buffers) and resource IDs (for texture/sampler).

* add a new test for parameter block

This test will fail if we don't have the fix in this PR.

For example:

```
cursor["scene"]["sceneCb"]["value"].setData(uint4{100, 100, 100, 100});
```

won't set anything because scene is a parameter block, however in
previous implementation, we didn't handle this correctly in ShaderObject
type layout and ShaderObject creation time.

* Add some comments on the new test

7489 of 13994 branches covered (53.52%)

Branch coverage included in aggregate %.

22849 of 34946 relevant lines covered (65.38%)

69566.22 hits per line