* [patch+8.0] release branch: Fix: --enable-werror
@ 2017-04-21 8:52 Jan Kratochvil
2017-04-21 14:04 ` Pedro Alves
2017-04-22 16:20 ` Sergio Durigan Junior
0 siblings, 2 replies; 5+ messages in thread
From: Jan Kratochvil @ 2017-04-21 8:52 UTC (permalink / raw)
To: gdb-patches; +Cc: Joel Brobecker, Sergio Durigan Junior
[-- Attachment #1: Type: text/plain, Size: 622 bytes --]
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?
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.
Sergio: Shouldn't it be also buildbot-tested?
Jan
[-- Attachment #2: 1 --]
[-- Type: text/plain, Size: 1127 bytes --]
gdb/ChangeLog
2017-04-21 Jan Kratochvil <jan.kratochvil@redhat.com>
* aarch64-tdep.c (selftests::aarch64_process_record_test): Make it #if
GDB_SELF_TEST.
* arm-tdep.c (selftests::arm_record_test): Likewise.
diff --git a/gdb/aarch64-tdep.c b/gdb/aarch64-tdep.c
index 28c2573..6113621 100644
--- a/gdb/aarch64-tdep.c
+++ b/gdb/aarch64-tdep.c
@@ -3040,10 +3040,12 @@ aarch64_dump_tdep (struct gdbarch *gdbarch, struct ui_file *file)
paddress (gdbarch, tdep->lowest_pc));
}
+#if GDB_SELF_TEST
namespace selftests
{
static void aarch64_process_record_test (void);
}
+#endif
/* Suppress warning from -Wmissing-prototypes. */
extern initialize_file_ftype _initialize_aarch64_tdep;
diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
index c8fabfb..9ac667f 100644
--- a/gdb/arm-tdep.c
+++ b/gdb/arm-tdep.c
@@ -9596,10 +9596,12 @@ arm_dump_tdep (struct gdbarch *gdbarch, struct ui_file *file)
(unsigned long) tdep->lowest_pc);
}
+#if GDB_SELF_TEST
namespace selftests
{
static void arm_record_test (void);
}
+#endif
extern initialize_file_ftype _initialize_arm_tdep; /* -Wmissing-prototypes */
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch+8.0] release branch: Fix: --enable-werror
2017-04-21 8:52 [patch+8.0] release branch: Fix: --enable-werror Jan Kratochvil
@ 2017-04-21 14:04 ` Pedro Alves
2017-04-21 14:18 ` [commit+8.0] " Jan Kratochvil
2017-04-22 16:20 ` Sergio Durigan Junior
1 sibling, 1 reply; 5+ messages in thread
From: Pedro Alves @ 2017-04-21 14:04 UTC (permalink / raw)
To: Jan Kratochvil, gdb-patches; +Cc: Joel Brobecker, Sergio Durigan Junior
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [commit+8.0] [patch+8.0] release branch: Fix: --enable-werror
2017-04-21 14:04 ` Pedro Alves
@ 2017-04-21 14:18 ` Jan Kratochvil
0 siblings, 0 replies; 5+ messages in thread
From: Jan Kratochvil @ 2017-04-21 14:18 UTC (permalink / raw)
To: Pedro Alves; +Cc: gdb-patches, Joel Brobecker, Sergio Durigan Junior
On Fri, 21 Apr 2017 16:03:50 +0200, Pedro Alves wrote:
> OK.
master:
0d4c07afb1e5e37e8e296858f8b14a3b4adf950b
8.0:
b2e445cf19788caa3640e8cfacab82995d2b1d13
Thanks,
Jan
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch+8.0] release branch: Fix: --enable-werror
2017-04-21 8:52 [patch+8.0] release branch: Fix: --enable-werror Jan Kratochvil
2017-04-21 14:04 ` Pedro Alves
@ 2017-04-22 16:20 ` Sergio Durigan Junior
2017-04-22 16:34 ` Jan Kratochvil
1 sibling, 1 reply; 5+ messages in thread
From: Sergio Durigan Junior @ 2017-04-22 16:20 UTC (permalink / raw)
To: Jan Kratochvil; +Cc: gdb-patches, Joel Brobecker
On Friday, April 21 2017, 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?
>
> 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.
>
> Sergio: Shouldn't it be also buildbot-tested?
Yes, it should. We have an Arch64 builder managed by Yao. It should
be possible to set up builders for the 8.0 branch.
> Jan
>
> gdb/ChangeLog
> 2017-04-21 Jan Kratochvil <jan.kratochvil@redhat.com>
>
> * aarch64-tdep.c (selftests::aarch64_process_record_test): Make it #if
> GDB_SELF_TEST.
> * arm-tdep.c (selftests::arm_record_test): Likewise.
>
> diff --git a/gdb/aarch64-tdep.c b/gdb/aarch64-tdep.c
> index 28c2573..6113621 100644
> --- a/gdb/aarch64-tdep.c
> +++ b/gdb/aarch64-tdep.c
> @@ -3040,10 +3040,12 @@ aarch64_dump_tdep (struct gdbarch *gdbarch, struct ui_file *file)
> paddress (gdbarch, tdep->lowest_pc));
> }
>
> +#if GDB_SELF_TEST
> namespace selftests
> {
> static void aarch64_process_record_test (void);
> }
> +#endif
>
> /* Suppress warning from -Wmissing-prototypes. */
> extern initialize_file_ftype _initialize_aarch64_tdep;
> diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
> index c8fabfb..9ac667f 100644
> --- a/gdb/arm-tdep.c
> +++ b/gdb/arm-tdep.c
> @@ -9596,10 +9596,12 @@ arm_dump_tdep (struct gdbarch *gdbarch, struct ui_file *file)
> (unsigned long) tdep->lowest_pc);
> }
>
> +#if GDB_SELF_TEST
> namespace selftests
> {
> static void arm_record_test (void);
> }
> +#endif
>
> extern initialize_file_ftype _initialize_arm_tdep; /* -Wmissing-prototypes */
>
>
--
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF 31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch+8.0] release branch: Fix: --enable-werror
2017-04-22 16:20 ` Sergio Durigan Junior
@ 2017-04-22 16:34 ` Jan Kratochvil
0 siblings, 0 replies; 5+ messages in thread
From: Jan Kratochvil @ 2017-04-22 16:34 UTC (permalink / raw)
To: Sergio Durigan Junior; +Cc: gdb-patches, Joel Brobecker
On Sat, 22 Apr 2017 18:20:06 +0200, Sergio Durigan Junior wrote:
> On Friday, April 21 2017, Jan Kratochvil wrote:
> > 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]
...
> > Sergio: Shouldn't it be also buildbot-tested?
>
> Yes, it should. We have an Arch64 builder managed by Yao. It should
> be possible to set up builders for the 8.0 branch.
FYI no specific arch is required for the builder, it even happened for me on
my x86_64. These were *-tdep.c files so --enable-targets=all is enough to
trigger it.
Sure more archs would make better coverage even for nat/* files.
Jan
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-04-22 16:34 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-21 8:52 [patch+8.0] release branch: Fix: --enable-werror Jan Kratochvil
2017-04-21 14:04 ` Pedro Alves
2017-04-21 14:18 ` [commit+8.0] " Jan Kratochvil
2017-04-22 16:20 ` Sergio Durigan Junior
2017-04-22 16:34 ` Jan Kratochvil
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).