diff options
| author | Samuel Pitoiset <[email protected]> | 2017-09-18 11:45:55 +0200 |
|---|---|---|
| committer | Samuel Pitoiset <[email protected]> | 2017-09-20 10:27:32 +0200 |
| commit | 60878dd00ceab6528c0cdb69fbd111470a789610 (patch) | |
| tree | 0323c28ccd2d912bfcb72c1ff75216d848057ce1 /src/amd/vulkan/radv_cmd_buffer.c | |
| parent | 505c2fea3a39af039fe6387642ad0e798b998d39 (diff) | |
radv: do not update the number of viewports in vkCmdSetViewport()
The Vulkan spec (1.0.61) says:
"The number of viewports used by a pipeline is still specified
by the viewportCount member of VkPipelineViewportStateCreateInfo."
So, the number of viewports is defined at pipeline creation
time and shouldn't be updated when they are set dynamically.
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Dave Airlie <[email protected]>
Diffstat (limited to 'src/amd/vulkan/radv_cmd_buffer.c')
| -rw-r--r-- | src/amd/vulkan/radv_cmd_buffer.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index d7b27cae27c..9724cc52940 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -2502,9 +2502,6 @@ void radv_CmdSetViewport( assert(firstViewport < MAX_VIEWPORTS); assert(total_count >= 1 && total_count <= MAX_VIEWPORTS); - if (cmd_buffer->state.dynamic.viewport.count < total_count) - cmd_buffer->state.dynamic.viewport.count = total_count; - memcpy(cmd_buffer->state.dynamic.viewport.viewports + firstViewport, pViewports, viewportCount * sizeof(*pViewports)); |