From f4a3bccf9452fca9d229c263aec8d124733a6d5d Mon Sep 17 00:00:00 2001 From: Iago Toral Quiroga Date: Thu, 7 Jul 2022 08:33:00 +0200 Subject: [PATCH] v3dv: remove obsolete comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit multop + umul24 can only be used to implement 32-bit multiplies, so for a full 64-bit result we always need to lower. Reviewed-by: Alejandro Piñeiro Part-of: --- src/broadcom/vulkan/v3dv_pipeline.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/broadcom/vulkan/v3dv_pipeline.c b/src/broadcom/vulkan/v3dv_pipeline.c index 82058c270e1..65a4507e4db 100644 --- a/src/broadcom/vulkan/v3dv_pipeline.c +++ b/src/broadcom/vulkan/v3dv_pipeline.c @@ -214,9 +214,6 @@ const nir_shader_compiler_options v3dv_nir_options = { .lower_pack_32_2x16 = true, .lower_pack_32_2x16_split = true, .lower_unpack_32_2x16_split = true, - /* FIXME: check if we can use multop + umul24 to implement mul2x32_64 - * without lowering. - */ .lower_mul_2x32_64 = true, .lower_fdiv = true, .lower_find_lsb = true,