From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 105195 invoked by alias); 12 Jun 2017 16:37:32 -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 105180 invoked by uid 89); 12 Jun 2017 16:37:31 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.2 spammy=HCc:D*ca X-HELO: mail-yw0-f169.google.com Received: from mail-yw0-f169.google.com (HELO mail-yw0-f169.google.com) (209.85.161.169) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 12 Jun 2017 16:37:30 +0000 Received: by mail-yw0-f169.google.com with SMTP id e142so29358065ywa.1 for ; Mon, 12 Jun 2017 09:37:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=mSNPxdFjbFhGXzOw0bTtC0Q0Vabi7MEaFeN05tPZyHk=; b=grcegpESok/o8xCz/CzjaWNwy7Gd3ZxLKbGphKAmmWjmjq9PutXaRgYAnfbJnzHW2E ZgWeHdnFUwInxSuOFkPNYi/oQB93zMvPmNkNRoz4nUClQfoj+zNlIOn9hSfUG+7nW9yh rBvyy4Loadiw9XKQyNzC0r0Bjvw1DQ/QmxQjn5npp8Kuqy0c7ZMFC8C8YFMnzDVCwEqG 5G1F82vsm8eFZqjcrvC4EEicDZDH/5XAXJqroZ/R2yd7WZEVO6UXw0SRJGwhL2DJBPcD BEjyUZ261QRFOUv/9mu+bOMiBVn9d96nBH4JSSwiYZQAp7jCZLrUdZhooho8zx9QWBYr CXyA== X-Gm-Message-State: AODbwcB1n06UZHRRD9VtzuBtizfEQ7Mw0WrgnQtqNWodLD4aq7Gs1NLO V46xTGJPcr883oH/dRgZvKbfPH7log== X-Received: by 10.129.72.75 with SMTP id v72mr26124665ywa.160.1497285453161; Mon, 12 Jun 2017 09:37:33 -0700 (PDT) MIME-Version: 1.0 Received: by 10.129.47.200 with HTTP; Mon, 12 Jun 2017 09:37:32 -0700 (PDT) In-Reply-To: <72d32638-ce7b-d362-5efd-84e8d89431d4@redhat.com> References: <1497124689-11842-1-git-send-email-simon.marchi@ericsson.com> <83tw3n5jyk.fsf@gnu.org> <86tw3labb0.fsf@gmail.com> <83a85d5l4n.fsf@gnu.org> <93eb64489ac9d53665a144ddf5a966d5@polymtl.ca> <72d32638-ce7b-d362-5efd-84e8d89431d4@redhat.com> From: Andrew Pinski Date: Mon, 12 Jun 2017 16:37:00 -0000 Message-ID: Subject: Re: [PATCH 0/5] Remove a few hurdles of compiling with clang To: Pedro Alves Cc: Simon Marchi , Eli Zaretskii , Yao Qi , Simon Marchi , "gdb-patches@sourceware.org" Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2017-06/txt/msg00332.txt.bz2 On Mon, Jun 12, 2017 at 9:35 AM, Pedro Alves wrote: > On 06/12/2017 05:23 PM, Andrew Pinski wrote: >> On Mon, Jun 12, 2017 at 8:54 AM, Simon Marchi wrote: > >>> - gdb: Add -Wno-mismatched-tags: We already have a system that tests which >>> warning flags are supported by the current compiler, so this flag will not >>> be included in the builds with GCC. So it's neutral for GCC, and improves >>> the situation for Clang with almost no effort. >> >> This warning is a bug in clang and really should not be warned about >> in either -Wall or -Wextra. I have been complaining about this since >> clang added this option. > > IIRC, the reason this warning exists is because Microsoft's compilers > mangle "struct" and "class" differently, so for projects that > want to be portable to that compiler, it's a helpful warning. > (Whether that should ever be part of -Wall is a separate matter...) > > I don't think we'd want to bend backwards to support MSVC > though. It's so non-conforming that it's scary. Disabling > that warning is the right thing to do, IMO. Why not have clang disable this warning by default instead? I am sorry but people who write C++ should understand that they are the same. Thanks, Andrew > > Thanks, > Pedro Alves >