public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Fix build errors in with clang in gdb.compile/compile-cplus.c
@ 2020-05-29 16:43 Gary Benson
0 siblings, 0 replies; only message in thread
From: Gary Benson @ 2020-05-29 16:43 UTC (permalink / raw)
To: gdb-cvs
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=678048e8079ace915052f3070b2df97bcaea58d2
commit 678048e8079ace915052f3070b2df97bcaea58d2
Author: Gary Benson <gbenson@redhat.com>
Date: Fri May 29 17:43:17 2020 +0100
Fix build errors in with clang in gdb.compile/compile-cplus.c
Clang fails to compile the file, with the following error:
fatal error: 'iostream' file not found
This prevents the following testcase from executing:
gdb.compile/compile-cplus.exp
The testcase sets additional_flags when building with GCC, which
this commit causes to also be set when building with clang. This
makes the testcase fail to build with a different error:
warning: treating 'c' input as 'c++' when in C++ mode, this behavior
is deprecated [-Wdeprecated]
so this commit adds -Wno-deprecated in two places to sidestep this.
Note that, while allowing the testcase to build, this commit reveals
failures when the testsuite is built using clang.
gdb/testsuite/ChangeLog:
* gdb.compile/compile-cplus.exp (additional_flags): Also
set when building with clang.
(additional_flags, srcfilesoptions): Pass -Wno-deprecated
when building with clang.
Diff:
---
gdb/testsuite/ChangeLog | 7 +++++++
gdb/testsuite/gdb.compile/compile-cplus.exp | 15 +++++++++++++--
2 files changed, 20 insertions(+), 2 deletions(-)
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 667dbfddec0..d4e7220b326 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,10 @@
+2020-05-29 Gary Benson <gbenson@redhat.com>
+
+ * gdb.compile/compile-cplus.exp (additional_flags): Also
+ set when building with clang.
+ (additional_flags, srcfilesoptions): Pass -Wno-deprecated
+ when building with clang.
+
2020-05-29 Gary Benson <gbenson@redhat.com>
* gdb.arch/i386-avx.exp (additional_flags): Also set when
diff --git a/gdb/testsuite/gdb.compile/compile-cplus.exp b/gdb/testsuite/gdb.compile/compile-cplus.exp
index cca5b205200..85b2f20a8fa 100644
--- a/gdb/testsuite/gdb.compile/compile-cplus.exp
+++ b/gdb/testsuite/gdb.compile/compile-cplus.exp
@@ -19,11 +19,16 @@ standard_testfile .c compile-shlib.c compile-constvar.S compile-nodebug.c
get_compiler_info
set options {}
-if [test_compiler_info gcc*] {
+if { [test_compiler_info gcc*] || [test_compiler_info clang*] } {
lappend options additional_flags=-g3
lappend options additional_flags=-std=gnu++11
lappend options c++
}
+if [test_compiler_info clang*] {
+ # Treating C input as C++ is deprecated in Clang, so
+ # the build will fail without disabling -Wdeprecated.
+ lappend options additional_flags=-Wno-deprecated
+}
if { ![istarget x86_64-*-* ] || ![is_lp64_target] } {
verbose "Skipping x86_64 LOC_CONST test."
@@ -34,7 +39,13 @@ set srcfilesoptions [list ${srcfile} ${options}]
if { $srcfile3 != "" } {
lappend srcfilesoptions $srcfile3 ${options}
}
-lappend srcfilesoptions $srcfile4 "nodebug c++"
+set srcfile4options "nodebug c++"
+if [test_compiler_info clang*] {
+ # Treating C input as C++ is deprecated in Clang, so
+ # the build will fail without disabling -Wdeprecated.
+ set srcfile4options "$srcfile4options additional_flags=-Wno-deprecated"
+}
+lappend srcfilesoptions $srcfile4 $srcfile4options
if { [eval build_executable_from_specs ${testfile}.exp $testfile {$options} ${srcfilesoptions}] } {
return -1
}
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-05-29 16:43 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-29 16:43 [binutils-gdb] Fix build errors in with clang in gdb.compile/compile-cplus.c Gary Benson
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).