v3dv: remove obsolete comment
authorIago Toral Quiroga <itoral@igalia.com>
Thu, 7 Jul 2022 06:33:00 +0000 (08:33 +0200)
committerMarge Bot <emma+marge@anholt.net>
Thu, 7 Jul 2022 09:16:24 +0000 (09:16 +0000)
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 <apinheiro@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17372>

src/broadcom/vulkan/v3dv_pipeline.c

index 82058c2..65a4507 100644 (file)
@@ -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,