Skip to content

Commit

Permalink
More fixups for Vulkan parameter block bindings (#309)
Browse files Browse the repository at this point in the history
I'm adding a small cross-compilation test to try to make sure that we are testing the binding generation for GLSL output. We probably still need a more complex test that uses multiple blocks, plus variables not in a block.

The big changes here are:

- Change the `containerTypeLayout` field to a `containerVarLayout` in the `ParameterGroupTypeLayout`, so that we can store the base offsets for the fields in a uniform fashion (even though these will all be zero).

- Switch the emit logic to carefully use either the container or element var layout depending on what they are emitting bindings for. This involved adding something akin to the "reflection path" notion that Falcor has to use, but only for the emit step.
  • Loading branch information
Tim Foley authored Dec 14, 2017
1 parent 6d61421 commit 4137f9d
Show file tree
Hide file tree
Showing 6 changed files with 303 additions and 138 deletions.
Loading

0 comments on commit 4137f9d

Please sign in to comment.