diff mbox series

vec_perm_const hook -- Fix build failure in ARM backend

Message ID CAAgBjMm-+xcWnrvn1uKug+=t8pCYEo9QH6TrJqxqyZ_p=57paQ@mail.gmail.com
State New
Headers show
Series vec_perm_const hook -- Fix build failure in ARM backend | expand

Commit Message

Prathamesh Kulkarni May 27, 2022, 7:11 a.m. UTC
Hi,
I forgot to adjust prototype for arm_vectorize_vec_perm_const, which,
resulted in following
build error:

# 00:05:33 make[3]: [Makefile:1787:
armv8l-unknown-linux-gnueabihf/bits/largefile-config.h] Error 1
(ignored)
# 00:10:53 /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gcc.git~master/gcc/config/arm/arm.cc:299:13:
error: ‘bool arm_vectorize_vec_perm_const(machine_mode, rtx, rtx, rtx,
const vec_perm_indices&)’ declared ‘static’ but never defined
[-Werror=unused-function]
# 00:12:22 make[3]: *** [Makefile:2418: arm.o] Error 1
# 00:23:34 make[2]: *** [Makefile:5005: all-stage2-gcc] Error 2
# 00:23:34 make[1]: *** [Makefile:25739: stage2-bubble] Error 2
# 00:23:34 make: *** [Makefile:1072: all] Error 2

https://gcc.gnu.org/pipermail/gcc-regression/2022-May/076645.html

The attached patch fixes it.
OK to commit ?

Thanks,
Prathamesh

Comments

Christophe Lyon May 30, 2022, 7:34 a.m. UTC | #1
Hi Prathamesh,


On 5/27/22 09:11, Prathamesh Kulkarni via Gcc-patches wrote:
> Hi,
> I forgot to adjust prototype for arm_vectorize_vec_perm_const, which,
> resulted in following
> build error:
> 
> # 00:05:33 make[3]: [Makefile:1787:
> armv8l-unknown-linux-gnueabihf/bits/largefile-config.h] Error 1
> (ignored)
> # 00:10:53 /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gcc.git~master/gcc/config/arm/arm.cc:299:13:
> error: ‘bool arm_vectorize_vec_perm_const(machine_mode, rtx, rtx, rtx,
> const vec_perm_indices&)’ declared ‘static’ but never defined
> [-Werror=unused-function]
> # 00:12:22 make[3]: *** [Makefile:2418: arm.o] Error 1
> # 00:23:34 make[2]: *** [Makefile:5005: all-stage2-gcc] Error 2
> # 00:23:34 make[1]: *** [Makefile:25739: stage2-bubble] Error 2
> # 00:23:34 make: *** [Makefile:1072: all] Error 2
> 
> https://gcc.gnu.org/pipermail/gcc-regression/2022-May/076645.html
> 
> The attached patch fixes it.
> OK to commit ?
> 

Doesn't this count as "obvious" ?

Thanks,

Christophe

> Thanks,
> Prathamesh
Prathamesh Kulkarni May 30, 2022, 7:59 a.m. UTC | #2
On Mon, 30 May 2022 at 13:04, Christophe Lyon <christophe.lyon@arm.com> wrote:
>
> Hi Prathamesh,
>
>
> On 5/27/22 09:11, Prathamesh Kulkarni via Gcc-patches wrote:
> > Hi,
> > I forgot to adjust prototype for arm_vectorize_vec_perm_const, which,
> > resulted in following
> > build error:
> >
> > # 00:05:33 make[3]: [Makefile:1787:
> > armv8l-unknown-linux-gnueabihf/bits/largefile-config.h] Error 1
> > (ignored)
> > # 00:10:53 /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gcc.git~master/gcc/config/arm/arm.cc:299:13:
> > error: ‘bool arm_vectorize_vec_perm_const(machine_mode, rtx, rtx, rtx,
> > const vec_perm_indices&)’ declared ‘static’ but never defined
> > [-Werror=unused-function]
> > # 00:12:22 make[3]: *** [Makefile:2418: arm.o] Error 1
> > # 00:23:34 make[2]: *** [Makefile:5005: all-stage2-gcc] Error 2
> > # 00:23:34 make[1]: *** [Makefile:25739: stage2-bubble] Error 2
> > # 00:23:34 make: *** [Makefile:1072: all] Error 2
> >
> > https://gcc.gnu.org/pipermail/gcc-regression/2022-May/076645.html
> >
> > The attached patch fixes it.
> > OK to commit ?
> >
>
> Doesn't this count as "obvious" ?
OK, pushed.

Thanks,
Prathamesh
>
> Thanks,
>
> Christophe
>
> > Thanks,
> > Prathamesh
diff mbox series

Patch

diff --git a/gcc/config/arm/arm.cc b/gcc/config/arm/arm.cc
index 70c2d50f0cc..7ecf7b74ffa 100644
--- a/gcc/config/arm/arm.cc
+++ b/gcc/config/arm/arm.cc
@@ -296,8 +296,8 @@  static int arm_cortex_a5_branch_cost (bool, bool);
 static int arm_cortex_m_branch_cost (bool, bool);
 static int arm_cortex_m7_branch_cost (bool, bool);
 
-static bool arm_vectorize_vec_perm_const (machine_mode, rtx, rtx, rtx,
-					  const vec_perm_indices &);
+static bool arm_vectorize_vec_perm_const (machine_mode, machine_mode, rtx, rtx,
+					  rtx, const vec_perm_indices &);
 
 static bool aarch_macro_fusion_pair_p (rtx_insn*, rtx_insn*);