From 56aa1ac74bc73ba18b2df5a5fdf908e8c12b4995 Mon Sep 17 00:00:00 2001
From: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Date: Mon, 8 Jul 2024 17:50:18 +0200
Subject: [PATCH] radv: use ac_is_reduction_mode_supported()

This reduces the number of formats that support filter min/max
reduction on <= GFX8 due to some hw limitations.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/30074>
---
 src/amd/vulkan/radv_formats.c | 87 +----------------------------------
 1 file changed, 1 insertion(+), 86 deletions(-)

diff --git a/src/amd/vulkan/radv_formats.c b/src/amd/vulkan/radv_formats.c
index 2a9c3078ce4..904c47d77c9 100644
--- a/src/amd/vulkan/radv_formats.c
+++ b/src/amd/vulkan/radv_formats.c
@@ -250,92 +250,7 @@ radv_is_zs_format_supported(VkFormat format)
 static bool
 radv_is_filter_minmax_format_supported(const struct radv_physical_device *pdev, VkFormat format)
 {
-   enum amd_gfx_level gfx_level = pdev->info.gfx_level;
-
-   switch (format) {
-   case VK_FORMAT_R4G4_UNORM_PACK8:
-   case VK_FORMAT_R4G4B4A4_UNORM_PACK16:
-   case VK_FORMAT_A4R4G4B4_UNORM_PACK16_EXT:
-   case VK_FORMAT_B4G4R4A4_UNORM_PACK16:
-   case VK_FORMAT_A4B4G4R4_UNORM_PACK16_EXT:
-   case VK_FORMAT_R5G6B5_UNORM_PACK16:
-   case VK_FORMAT_B5G6R5_UNORM_PACK16:
-   case VK_FORMAT_R5G5B5A1_UNORM_PACK16:
-   case VK_FORMAT_B5G5R5A1_UNORM_PACK16:
-   case VK_FORMAT_A1R5G5B5_UNORM_PACK16:
-   case VK_FORMAT_A1B5G5R5_UNORM_PACK16_KHR:
-   case VK_FORMAT_R8_UNORM:
-   case VK_FORMAT_A8_UNORM_KHR:
-   case VK_FORMAT_R8_SNORM:
-   case VK_FORMAT_R8_SRGB:
-   case VK_FORMAT_R8G8_UNORM:
-   case VK_FORMAT_R8G8_SNORM:
-   case VK_FORMAT_R8G8_SRGB:
-   case VK_FORMAT_R8G8B8A8_UNORM:
-   case VK_FORMAT_R8G8B8A8_SNORM:
-   case VK_FORMAT_R8G8B8A8_SRGB:
-   case VK_FORMAT_B8G8R8A8_UNORM:
-   case VK_FORMAT_B8G8R8A8_SNORM:
-   case VK_FORMAT_B8G8R8A8_SRGB:
-   case VK_FORMAT_A8B8G8R8_UNORM_PACK32:
-   case VK_FORMAT_A8B8G8R8_SNORM_PACK32:
-   case VK_FORMAT_A8B8G8R8_SRGB_PACK32:
-   case VK_FORMAT_A2R10G10B10_UNORM_PACK32:
-   case VK_FORMAT_A2B10G10R10_UNORM_PACK32:
-   case VK_FORMAT_R16_UNORM:
-   case VK_FORMAT_R16_SNORM:
-   case VK_FORMAT_R16_SFLOAT:
-   case VK_FORMAT_R16G16_UNORM:
-   case VK_FORMAT_R16G16_SNORM:
-   case VK_FORMAT_R16G16_SFLOAT:
-   case VK_FORMAT_R16G16B16A16_UNORM:
-   case VK_FORMAT_R16G16B16A16_SNORM:
-   case VK_FORMAT_R16G16B16A16_SFLOAT:
-   case VK_FORMAT_R32_SFLOAT:
-   case VK_FORMAT_R32G32_SFLOAT:
-   case VK_FORMAT_R32G32B32_SFLOAT:
-   case VK_FORMAT_R32G32B32A32_SFLOAT:
-   case VK_FORMAT_B10G11R11_UFLOAT_PACK32:
-   case VK_FORMAT_D16_UNORM:
-   case VK_FORMAT_X8_D24_UNORM_PACK32:
-   case VK_FORMAT_D32_SFLOAT:
-   case VK_FORMAT_D16_UNORM_S8_UINT:
-   case VK_FORMAT_D24_UNORM_S8_UINT:
-   case VK_FORMAT_D32_SFLOAT_S8_UINT:
-      return true;
-   case VK_FORMAT_R8_UINT:
-   case VK_FORMAT_R8_SINT:
-   case VK_FORMAT_R8G8_UINT:
-   case VK_FORMAT_R8G8_SINT:
-   case VK_FORMAT_R8G8B8A8_UINT:
-   case VK_FORMAT_R8G8B8A8_SINT:
-   case VK_FORMAT_B8G8R8A8_UINT:
-   case VK_FORMAT_B8G8R8A8_SINT:
-   case VK_FORMAT_A8B8G8R8_UINT_PACK32:
-   case VK_FORMAT_A8B8G8R8_SINT_PACK32:
-   case VK_FORMAT_A2R10G10B10_UINT_PACK32:
-   case VK_FORMAT_A2B10G10R10_UINT_PACK32:
-   case VK_FORMAT_R16_UINT:
-   case VK_FORMAT_R16_SINT:
-   case VK_FORMAT_R16G16_UINT:
-   case VK_FORMAT_R16G16_SINT:
-   case VK_FORMAT_R16G16B16A16_UINT:
-   case VK_FORMAT_R16G16B16A16_SINT:
-   case VK_FORMAT_R32_UINT:
-   case VK_FORMAT_R32_SINT:
-   case VK_FORMAT_R32G32_UINT:
-   case VK_FORMAT_R32G32_SINT:
-   case VK_FORMAT_R32G32B32_UINT:
-   case VK_FORMAT_R32G32B32_SINT:
-   case VK_FORMAT_R32G32B32A32_UINT:
-   case VK_FORMAT_R32G32B32A32_SINT:
-   case VK_FORMAT_S8_UINT:
-      return gfx_level >= GFX9;
-   case VK_FORMAT_E5B9G9R9_UFLOAT_PACK32:
-      return gfx_level >= GFX7;
-   default:
-      return false;
-   }
+   return ac_is_reduction_mode_supported(&pdev->info, vk_format_to_pipe_format(format), false);
 }
 
 bool