fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2025-12-24 11:00:11 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

pvr: Dedup a check with pvr_is_render_area_tile_aligned()

Browse source 
Signed-off-by: Karmjit Mahil <Karmjit.Mahil@imgtec.com>
Reviewed-by: Frank Binns <frank.binns@imgtec.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22657>
This commit is contained in:
Karmjit Mahil 2023-02-10 16:00:04 +00:00
parent c57a49c234
commit 10c0c68f47
1 changed files with 2 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/imagination/vulkan/pvr_cmd_buffer.c
View file

@ -2717,8 +2717,6 @@ static void pvr_perform_start_of_render_attachment_clear(
VkImageAspectFlags image_aspect;
struct pvr_image_view *iview;
uint32_t view_idx;
uint32_t height;
uint32_t width;
if (is_depth_stencil) {
bool stencil_clear;
@ -2749,8 +2747,6 @@ static void pvr_perform_start_of_render_attachment_clear(
}
iview = info->attachments[view_idx];
width = iview->vk.extent.width;
height = iview->vk.extent.height;
/* FIXME: It would be nice if this function and pvr_sub_cmd_gfx_job_init()
* were doing the same check (even if it's just an assert) to determine if a
@ -2759,9 +2755,8 @@ static void pvr_perform_start_of_render_attachment_clear(
/* If this is single-layer fullscreen, we already do the clears in
* pvr_sub_cmd_gfx_job_init().
*/
if (info->render_area.offset.x == 0 && info->render_area.offset.y == 0 &&
info->render_area.extent.width == width &&
info->render_area.extent.height == height && framebuffer->layers == 1) {
if (pvr_is_render_area_tile_aligned(cmd_buffer, iview) &&
framebuffer->layers == 1) {
return;
}