This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [AArch64/AArch64-4.7] Fix warning - No previous prototype for aarch64_init_simd_builtins.
- From: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>
- To: James Greenhalgh <james dot greenhalgh at arm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, marcus dot shawcroft at arm dot com
- Date: Thu, 28 Feb 2013 16:23:12 +0000
- Subject: Re: [AArch64/AArch64-4.7] Fix warning - No previous prototype for aarch64_init_simd_builtins.
- References: <1362044092-7629-1-git-send-email-james.greenhalgh@arm.com>
OK
On 28 February 2013 09:34, James Greenhalgh <james.greenhalgh@arm.com> wrote:
>
> Hi,
>
> aarch64_init_simd_buitlins is not declared with a prototype, but it
> doesn't need one as it should be declared static.
>
> This patch fixes the warning:
>
> config/aarch64/aarch64-builtins.c:314:1: warning: no previous prototype for ‘aarch64_init_simd_builtins’ [-Wmissing-prototypes]
>
> Which is hidden when building with g++, but looks like:
>
> config/aarch64/aarch64-builtins.c:313:1: warning: no previous declaration for ‘aarch64_init_simd_builtins’ [-Wmissing-declarations]
>
> On Trunk.
>
> Regression tested with no regressions on aarch64-none-elf.
>
> OK to apply to trunk and aarch64-4.7-branch?
>
> Thanks,
> James Greenhalgh
>
> ---
> gcc/
>
> 2013-02-28 James Greenhalgh <james.greenhalgh@arm.com>
>
> * config/aarch64/aarch64-builtins.c
> (aarch64_init_simd_builtins): Make static.