diff options
| author | Jason Ekstrand <jason.ekstrand@intel.com> | 2016-10-05 16:51:02 -0700 |
|---|---|---|
| committer | Jason Ekstrand <jason.ekstrand@intel.com> | 2016-10-06 16:52:31 -0700 |
| commit | b548fdbed5c16f5c0dfc26f65b0037a85c567735 (patch) | |
| tree | 083e4f04c8b71041ab7aa80701964e4234d82aae /src/intel/vulkan/genX_cmd_buffer.c | |
| parent | fe4e276b02615b5db8acbf4c65fcfa68982e2e0f (diff) | |
| download | external_mesa3d-b548fdbed5c16f5c0dfc26f65b0037a85c567735.zip external_mesa3d-b548fdbed5c16f5c0dfc26f65b0037a85c567735.tar.gz external_mesa3d-b548fdbed5c16f5c0dfc26f65b0037a85c567735.tar.bz2 | |
anv/cmd_buffer: Don't call set_subpass in a secondary
Initially, we had intended set_subpass to be an interesting function that
did whatever (presumably a lot) setup we needed for a subpass. In reality,
it just sets a pointer and a dirty bit and then emits depth and stencil
state. When we call BeginCommandBuffer on a secondary, there's no point in
setting depth and stencil state since it will already be set by the
primary. Instead, the only thing we need to do at the start of a secondary
is set the subpass pointer and the dirty bit.
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Nanley Chery <nanley.g.chery@intel.com>
Diffstat (limited to 'src/intel/vulkan/genX_cmd_buffer.c')
| -rw-r--r-- | src/intel/vulkan/genX_cmd_buffer.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index 5fbf3a2..02f81b1 100644 --- a/src/intel/vulkan/genX_cmd_buffer.c +++ b/src/intel/vulkan/genX_cmd_buffer.c @@ -1296,10 +1296,7 @@ cmd_buffer_emit_depth_stencil(struct anv_cmd_buffer *cmd_buffer) anv_batch_emit(&cmd_buffer->batch, GENX(3DSTATE_CLEAR_PARAMS), cp); } -/** - * @see anv_cmd_buffer_set_subpass() - */ -void +static void genX(cmd_buffer_set_subpass)(struct anv_cmd_buffer *cmd_buffer, struct anv_subpass *subpass) { |