Hi Richard,
> >
> > 2020-01-18 Tamar Christina <tamar.christ...@arm.com>
> >
> > * config/aarch64/aarch64-sve-builtins-base.cc
> (memory_vector_mode):
> > Mark parameter unused.
>
> Thanks for the quick fix.
>
> Sorry for indulging a personal preference, but for things like this, I think
> it's
> nicer to drop the parameter name rather than add ATTRIBUTE_UNUSED.
> That's the "style" used elsewhere in the file and also keeps the line length
> under 80 chars.
Ah, I did not know this trick.. Thanks!
Tamar.
>
> Tested on aarch64-linux-gnu and applied.
>
> Richard
>
>
> 2020-01-20 Richard Sandiford <richard.sandif...@arm.com>
>
> gcc/
> * config/aarch64/aarch64-sve-builtins-base.cc
> (svld1ro_impl::memory_vector_mode): Remove parameter name.
> ---
> gcc/config/aarch64/aarch64-sve-builtins-base.cc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/aarch64/aarch64-sve-builtins-base.cc
> b/gcc/config/aarch64/aarch64-sve-builtins-base.cc
> index b48932f6a4f..26f0b8c6652 100644
> --- a/gcc/config/aarch64/aarch64-sve-builtins-base.cc
> +++ b/gcc/config/aarch64/aarch64-sve-builtins-base.cc
> @@ -1206,7 +1206,7 @@ class svld1ro_impl : public load_replicate {
> public:
> machine_mode
> - memory_vector_mode (const function_instance &fi ATTRIBUTE_UNUSED)
> const OVERRIDE
> + memory_vector_mode (const function_instance &) const OVERRIDE
> {
> return OImode;
> }
