From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 82399 invoked by alias); 17 Jun 2017 21:23:56 -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 82386 invoked by uid 89); 17 Jun 2017 21:23:55 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-6.1 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_2,SPF_HELO_PASS,SPF_SOFTFAIL autolearn=ham version=3.3.2 spammy=fruits, hanging, Hx-languages-length:1207 X-HELO: simark.ca Received: from simark.ca (HELO simark.ca) (158.69.221.121) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 17 Jun 2017 21:23:52 +0000 Received: by simark.ca (Postfix, from userid 33) id A8B451E55D; Sat, 17 Jun 2017 17:23:55 -0400 (EDT) To: Simon Marchi Subject: Re: [PATCH 0/5] Remove a few hurdles of compiling with clang X-PHP-Originating-Script: 33:rcube.php MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Sat, 17 Jun 2017 21:23:00 -0000 From: Simon Marchi Cc: gdb-patches@sourceware.org In-Reply-To: <1497124689-11842-1-git-send-email-simon.marchi@ericsson.com> References: <1497124689-11842-1-git-send-email-simon.marchi@ericsson.com> Message-ID: <0cac219c61a95a84c76e6c78b9465f7e@polymtl.ca> X-Sender: simon.marchi@polymtl.ca User-Agent: Roundcube Webmail/1.2.5 X-IsSubscribed: yes X-SW-Source: 2017-06/txt/msg00476.txt.bz2 On 2017-06-10 21:58, Simon Marchi wrote: > It is currently possible to build with clang by jumping through a few > hoops and > compiling without -Werror, but it is not pretty. There is a _ton_ of > warnings. > clang often gives some good and relevant warnings (e.g. [1]), so it > would be > useful to get the number to a reasonnable level to be able to see those > that > are actually relevant. I started to work on the lowest hanging fruits > and the > changes that should not be too controversial. > > [1] https://sourceware.org/ml/gdb-patches/2017-06/msg00252.html > > Simon Marchi (5): > gdb: Pass -x c++ to the compiler > gdb: Use -Werror when checking for (un)supported warning flags > gdb: Add -Wno-mismatched-tags > linux-low: Remove usage of "register" keyword > Add ATTRIBUTE_PRINTF to trace_start_error > > gdb/Makefile.in | 2 +- > gdb/configure | 7 ++++--- > gdb/gdbserver/Makefile.in | 2 +- > gdb/gdbserver/configure | 13 +++++++------ > gdb/gdbserver/linux-low.c | 16 ++++++++-------- > gdb/nat/fork-inferior.h | 2 +- > gdb/warning.m4 | 7 ++++--- > 7 files changed, 26 insertions(+), 23 deletions(-) This is now pushed in. Simon