From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 91135 invoked by alias); 18 Nov 2015 16:40:08 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 91107 invoked by uid 89); 18 Nov 2015 16:40:07 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Wed, 18 Nov 2015 16:40:05 +0000 Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) by mx1.redhat.com (Postfix) with ESMTPS id 5E5AD461DA; Wed, 18 Nov 2015 16:40:04 +0000 (UTC) Received: from brno.lan (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx13.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id tAIGe2Q7030267; Wed, 18 Nov 2015 11:40:03 -0500 From: Pedro Alves To: gdb-patches@sourceware.org Cc: Simon Marchi Subject: [PATCH 0/6] [C++] Drop -fpermissive hack, enable -Werror Date: Wed, 18 Nov 2015 16:40:00 -0000 Message-Id: <1447864802-24016-1-git-send-email-palves@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-SW-Source: 2015-11/txt/msg00364.txt.bz2 The first 4 patches finally make GDB build cleanly in C++, at least on x86_64 GNU/Linux and x86_64 mingw. At this point, I think we should drop the -fpermissive hack and default to -Werror in C++ mode too, which is what the last two patches do. This lets the buildbot catch C++ build regressions promptly -- we already have a Fedora buildslave building in C++ mode specifically for that, but it currently misses regressions around pointer casts and enum conversions exactly due to -fpermissive/-Wno-error. Other ports still need further C++ conversion work, though the build failures shouldn't be too many and should be mostly localized to host/target-specific code. Pedro Alves (5): [C++] remote.c: Avoid enum arithmetic [C++] linux-thread-db.c: dladdr cast [C++] breakpoint.c: "no memory" software watchpoints and enum casts [C++] Drop -fpermissive hack [C++] Default to -Werror in C++ mode too Simon Marchi (1): [C++] s390: Fix enum gdb_syscall conversion gdb/breakpoint.c | 49 ++++++++++++++++++++++++++++++++++------------ gdb/build-with-cxx.m4 | 3 +-- gdb/configure | 10 +++------- gdb/configure.ac | 7 ++----- gdb/gdbserver/configure | 6 ++---- gdb/gdbserver/configure.ac | 3 +-- gdb/linux-thread-db.c | 2 +- gdb/remote.c | 2 +- gdb/s390-linux-tdep.c | 41 ++++++++++++++++++++++---------------- gdb/target.c | 36 +++++++++++++++++++++++----------- gdb/target.h | 4 ++++ 11 files changed, 100 insertions(+), 63 deletions(-) -- 1.9.3