From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 54175 invoked by alias); 10 Aug 2016 14:45:18 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 54155 invoked by uid 89); 10 Aug 2016 14:45:17 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy=H*i:CAESRpQCB-RnqJV, H*MI:CAESRpQCB-RnqJV, H*f:sk:zmGL_Nj, H*f:CAESRpQCB-RnqJV X-HELO: mail-wm0-f65.google.com Received: from mail-wm0-f65.google.com (HELO mail-wm0-f65.google.com) (74.125.82.65) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Wed, 10 Aug 2016 14:45:03 +0000 Received: by mail-wm0-f65.google.com with SMTP id i5so9994101wmg.2 for ; Wed, 10 Aug 2016 07:45:03 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:subject:to:references:cc:from:message-id :date:user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=VZOk3KHrEJgJf4//eeBBdyV+FNlm5UQmLslLmZrFcWo=; b=HBkZtVsfVwh/cCMv972lzmruK6STur4Fbf2YEKrSWzogqqg/2VEFetF7wGkMer/Bql /FK9vE+UqsFdOph9NDggKqdG2mkZckgpV1aepmqeJEfOMMhJb0wbt6nBzMwDxsj0glmk +rraQmPTCYsXyNaEXiOHrTtpP9uYVC67XbPe0+0rISRnVpVlcoxdqtfJJO6m5vVU30m8 KqpBlG5KzJU1SMUxqSlYm46jcU3i8eXJXVEliopExtuquKcKm1NOmkw2CuNx4icx1Mvh buPKiI9wEla9FzQsyy5NI9HoHG3q/64HeZoalK5EKytyR4mcvb0Ivf3jxrgSLVJyfnnV 1fUQ== X-Gm-Message-State: AEkooutcvLcEWvcfMYaD+Pi+TfPWarXoPt2+8Ysz9beCtYxVyT6gZYFbgHrZ/kIjvxx6dA== X-Received: by 10.28.50.3 with SMTP id y3mr3791112wmy.23.1470840300459; Wed, 10 Aug 2016 07:45:00 -0700 (PDT) Received: from [192.168.10.165] (94-39-152-88.adsl-ull.clienti.tiscali.it. [94.39.152.88]) by smtp.googlemail.com with ESMTPSA id o2sm17277683wjo.3.2016.08.10.07.44.57 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 10 Aug 2016 07:44:58 -0700 (PDT) Subject: Re: [PATCH v2] cpp/c: Add -Wexpansion-to-defined To: =?UTF-8?B?TWFudWVsIEzDs3Blei1JYsOhw7Fleg==?= References: <1470823595-1471-1-git-send-email-bonzini@gnu.org> <59051c81-0056-3e01-54a6-07b71307965c@gnu.org> Cc: Gcc Patch List From: Paolo Bonzini Message-ID: Date: Wed, 10 Aug 2016 14:45:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-SW-Source: 2016-08/txt/msg00827.txt.bz2 On 10/08/2016 16:42, Manuel López-Ibáñez wrote: > > > My only fear is that people not using -Wpedantic nor -pedantic-errors > > > expect that GNU extensions work. This is a GNU extension that defines > > > something that is undefined according to ISO. Enabling the warning > > > with -Wextra is just annoying those people who may not care about > > > other compilers. > > > > I think this warning falls in the same category as > > -Wshift-negative-value. (In fact I dislike -Wshift-negative-value a > > lot, and would put that one under -Wpedantic only). > > It is not the same category. One is compile-time UB and the other is > runtime UB. See: > https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01551.html > and https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01529.html Right---what I meant is it's the same kind of "annoying for people who expect that GNU extensions work" warning. Paolo