From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28983 invoked by alias); 8 Oct 2010 08:51:51 -0000 Received: (qmail 28967 invoked by uid 22791); 8 Oct 2010 08:51:50 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE X-Spam-Check-By: sourceware.org Received: from mx01.qsc.de (HELO mx01.qsc.de) (213.148.129.14) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 08 Oct 2010 08:51:45 +0000 Received: from [192.168.178.22] (port-92-204-102-2.dynamic.qsc.de [92.204.102.2]) by mx01.qsc.de (Postfix) with ESMTP id 73AF83CE13; Fri, 8 Oct 2010 10:51:41 +0200 (CEST) Message-ID: <4CAEDB9C.2060904@net-b.de> Date: Fri, 08 Oct 2010 08:56:00 -0000 From: Tobias Burnus User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.9) Gecko/20100914 SUSE/3.1.4 Thunderbird/3.1.4 MIME-Version: 1.0 To: FX CC: gcc-patches@gcc.gnu.org Subject: Re: [patch] Separate {OS,CPU}_CPP_BUILTINS macros into C-family and language-independent macros References: <4D88077B-3233-4D15-94E6-13BE58D29AD5@gmail.com> In-Reply-To: <4D88077B-3233-4D15-94E6-13BE58D29AD5@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-10/txt/msg00656.txt.bz2 On 10/07/2010 12:49 PM, FX wrote: > The patch moves this C-only code to specific TARGET_OS_CPP_BUILTINS_CFAMILY and TARGET_CPU_CPP_BUILTINS_CFAMILY macros. I manually audited all the targets in config/ twice, to check that I did not leave any code behind. I have bootstrapped and regtested an all-language compiler on x86_64-linux and i686-darwin. I have also built C and Fortran cross-compilers (make all-gcc&& make fortran) for the following targets (which cover all gcc/config/ files changed): > OK to commit? The Fortran changes are OK. For approval of gcc/c-family/, cppbuiltin.{c,h}, doc/tm.texi{,.in}, and config/* you need to find someone else ... Tobias > gcc/fortran/ > > 2010-10-07 Francois-Xavier Coudert > > PR fortran/42954 > * cpp.c (cpp_define_builtins): Call define_target_specific_builtins > instead of TARGET_CPU_CPP_BUILTINS, TARGET_OS_CPP_BUILTINS and > TARGET_OBJFMT_CPP_BUILTINS.