[patch+8.0] release branch: Fix: --enable-werror

Pedro Alves palves@redhat.com
Fri Apr 21 14:04:00 GMT 2017


On 04/21/2017 09:52 AM, Jan Kratochvil wrote:
> Hi,
> 
> gdb-8.0-branch
> ./configure --enable-werror --enable-targets=all
> aarch64-tdep.c:3045:13: error: ‘void selftests::aarch64_process_record_test()’ declared ‘static’ but never defined [-Werror=unused-function]
> arm-tdep.c:9601:13: error: ‘void selftests::arm_record_test()’ declared ‘static’ but never defined [-Werror=unused-function]
> 
> OK for check-in?

OK.

> 
> Please do not tell me release branches do not support --enable-werror as this
> is due to a sourcebase bug, not due to some unexpected compiler
> incompatibility.

Certainly.

> Sergio: Shouldn't it be also buildbot-tested?

Sounds like a good idea.

Thanks,
Pedro Alves



More information about the Gdb-patches mailing list