This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] Fix compile error with clang 3.8
- From: Simon Marchi <simark at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 21 Dec 2018 16:56:10 -0000
- Subject: [binutils-gdb] Fix compile error with clang 3.8
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=3dcfdc58656caa4a38e1fde73a07a19114347226
commit 3dcfdc58656caa4a38e1fde73a07a19114347226
Author: Dave Murphy <davem@devkitpro.org>
Date: Fri Dec 21 11:14:28 2018 -0500
Fix compile error with clang 3.8
When compiling with clang 3.8 (default clang version on Debian
Stretch, the current stable), we get errors like this:
CXX dtrace-probe.o
../../binutils-gdb/gdb/dtrace-probe.c:103:31: error: default initialization of an object of const type 'const dtrace_static_probe_ops' without a user-provided default constructor
const dtrace_static_probe_ops dtrace_static_probe_ops;
^
Silence them by value-initializing those objects. It's not necessary
with other compilers (later clang versions, gcc), but it shouldn't
hurt either.
Diff:
---
gdb/ChangeLog | 10 ++++++++++
gdb/dtrace-probe.c | 2 +-
gdb/probe.c | 2 +-
gdb/record-btrace.c | 2 +-
gdb/stap-probe.c | 2 +-
gdb/tui/tui-hooks.c | 2 +-
gdb/unittests/observable-selftests.c | 2 +-
7 files changed, 16 insertions(+), 6 deletions(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 28ab334..953bc17 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,13 @@
+2018-12-21 Dave Murphy <davem@devkitpro.org>
+
+ * dtrace-probe.c (dtrace_static_probe_ops): Explicit zero
+ initialise.
+ * probe.c (any_static_probe_ops): Ditto.
+ * record-btrace.c (record_btrace_thread_observer_token): Ditto.
+ * stap-probe.c (stap_static_probe_ops): Ditto.
+ * tui/tui-hooks.c (tui_observers_token): Ditto.
+ * unittests/observable-selftests.c (token1, token2, token3): Ditto.
+
2018-12-19 Andrew Burgess <andrew.burgess@embecosm.com>
* gdb/dummy-frame.c (default_dummy_id): Defined new function.
diff --git a/gdb/dtrace-probe.c b/gdb/dtrace-probe.c
index fa4e06e..0c75904 100644
--- a/gdb/dtrace-probe.c
+++ b/gdb/dtrace-probe.c
@@ -100,7 +100,7 @@ public:
/* DTrace static_probe_ops. */
-const dtrace_static_probe_ops dtrace_static_probe_ops;
+const dtrace_static_probe_ops dtrace_static_probe_ops {};
/* The following structure represents a dtrace probe. */
diff --git a/gdb/probe.c b/gdb/probe.c
index 1f3da21..5702afe 100644
--- a/gdb/probe.c
+++ b/gdb/probe.c
@@ -60,7 +60,7 @@ public:
/* Static operations associated with a generic probe. */
-const any_static_probe_ops any_static_probe_ops;
+const any_static_probe_ops any_static_probe_ops {};
/* A helper for parse_probes that decodes a probe specification in
SEARCH_PSPACE. It appends matching SALs to RESULT. */
diff --git a/gdb/record-btrace.c b/gdb/record-btrace.c
index 1ca0176..92c0821 100644
--- a/gdb/record-btrace.c
+++ b/gdb/record-btrace.c
@@ -143,7 +143,7 @@ static record_btrace_target record_btrace_ops;
/* Token associated with a new-thread observer enabling branch tracing
for the new thread. */
-static const gdb::observers::token record_btrace_thread_observer_token;
+static const gdb::observers::token record_btrace_thread_observer_token {};
/* Memory access types used in set/show record btrace replay-memory-access. */
static const char replay_memory_access_read_only[] = "read-only";
diff --git a/gdb/stap-probe.c b/gdb/stap-probe.c
index ed7e1a0..b79cb15 100644
--- a/gdb/stap-probe.c
+++ b/gdb/stap-probe.c
@@ -119,7 +119,7 @@ public:
/* SystemTap static_probe_ops. */
-const stap_static_probe_ops stap_static_probe_ops;
+const stap_static_probe_ops stap_static_probe_ops {};
class stap_probe : public probe
{
diff --git a/gdb/tui/tui-hooks.c b/gdb/tui/tui-hooks.c
index efa02e2..3a9bff3 100644
--- a/gdb/tui/tui-hooks.c
+++ b/gdb/tui/tui-hooks.c
@@ -205,7 +205,7 @@ tui_normal_stop (struct bpstats *bs, int print_frame)
/* Token associated with observers registered while TUI hooks are
installed. */
-static const gdb::observers::token tui_observers_token;
+static const gdb::observers::token tui_observers_token {};
/* Attach or detach a single observer, according to ATTACH. */
diff --git a/gdb/unittests/observable-selftests.c b/gdb/unittests/observable-selftests.c
index 6f2f7e8..903baa7 100644
--- a/gdb/unittests/observable-selftests.c
+++ b/gdb/unittests/observable-selftests.c
@@ -70,7 +70,7 @@ run_tests ()
attached. */
notify_check_counters (0, 0, 0);
- const gdb::observers::token token1, token2, token3;
+ const gdb::observers::token token1 {}, token2 {} , token3 {};
/* Now, attach one observer, and send a notification. */
test_notification.attach (&test_second_notification_function, token2);