From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13851 invoked by alias); 6 Nov 2010 00:03:14 -0000 Received: (qmail 13819 invoked by uid 22791); 6 Nov 2010 00:03:12 -0000 X-SWARE-Spam-Status: No, hits=-2.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from out02.roch.ny.frontiernet.net (HELO out02.roch.ny.frontiernet.net) (66.133.183.227) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sat, 06 Nov 2010 00:03:05 +0000 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: AvsEAE441ExHc96u/2dsb2JhbAChcbtehUgEhFiLCg Received: from relay01.roch.ny.frontiernet.net ([66.133.182.164]) by out02.roch.ny.frontiernet.net with ESMTP; 06 Nov 2010 00:03:03 +0000 X-Previous-IP: 71.115.222.174 Received: from quava.localdomain (pool-71-115-222-174.spknwa.dsl-w.verizon.net [71.115.222.174]) by relay01.roch.ny.frontiernet.net (Postfix) with ESMTPA id E4DB03FED6; Sat, 6 Nov 2010 00:03:00 +0000 (UTC) Message-ID: <4CD49B1A.20603@frontier.com> Date: Sat, 06 Nov 2010 00:06:00 -0000 From: Jerry DeLisle User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.12) Gecko/20101027 Fedora/3.1.6-1.fc13 Thunderbird/3.1.6 MIME-Version: 1.0 To: FX CC: gcc-patches@gcc.gnu.org, gfortran List , Michael Meissner , Rainer Orth , chertykov@gmail.com, aesok@post.ru, eric.weddington@atmel.com Subject: Re: [patch] Separate {OS,CPU}_CPP_BUILTINS macros into C-family and language-independent macros References: <1D974441-C894-4F24-A0D1-7B693044B2BF@gmail.com> In-Reply-To: <1D974441-C894-4F24-A0D1-7B693044B2BF@gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit 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 X-SW-Source: 2010-11/txt/msg00606.txt.bz2 On 11/03/2010 02:26 PM, FX wrote: > See below for the description of the patch; this take n+1 incorporates suggestions by Michael Meissner and Rainer Orth, with the following exceptions: > > -- As avr.c includes new headers (cpplib.h and cppbuiltin.h), I wanted to update its dependencies in the target configuration file (t-avr), but to my surprise, this file doesn't contain any mention of avr.o. In fact, I failed to find any by grepping the whole tree, so I'm a little unsure how this can work (and apparently it does). Could the maintainers help me here? > > -- Michael is worried about splitting all this stuff: > >>> Unless I don't see what you're talking about, this is not possible: the Fortran front-end is not linked with the *-c.c files, only with *.c, so the language-independent functions need to be moved there. >> >> Yes, but if fortran is going to use the preprocessor, maybe they should be >> linked in. I'm just worried about the possibility of a macro getting lost in >> the movement. > > > Well, we do need some separation between CPP built-in macros that are only relevant for C-family languages (for example, because they depend on flag variables that are only declared for C) and the others. Now that the job is done, was discussed beforehand, and it fixes the regression experienced by Fortran, I'd be glad to just have it work. > > > Tested as indicated below, OK to commit? (needs a global reviewer; pretty pretty please, don't let it bitrot again) I'd like to highlight that it's nothing conceptualy major, and it'll be easy to fix any eventual fallout. > > FX I have reviewed this patch and I have to agree with FX. There is nothing here that makes these workings more obfuscated then they already are and I think the splitting actually makes it easier to understand, ie less obfuscated. Considering the lessor evil and that it passes testing, and I would like to encourage this patch be approved. I would like to see specific comments as to why not if not, so we can close on this. Regards, Jerry