This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [patch, fortran] PR18428: use libcpp for preprocessing (review of C/C++ and fortran maintainers needed)
- From: "Danny Smith" <dansmister at gmail dot com>
- To: "Tobias Burnus" <burnus at net-b dot de>
- Cc: "Daniel Franke" <franke dot daniel at gmail dot com>, fortran at gcc dot gnu dot org, FX <fxcoudert at gmail dot com>, gcc-patches at gcc dot gnu dot org, tromey at redhat dot com
- Date: Mon, 26 May 2008 20:22:25 +1200
- Subject: Re: [patch, fortran] PR18428: use libcpp for preprocessing (review of C/C++ and fortran maintainers needed)
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; bh=ZEgrvZDN8i5u4M4W4U1rEumgtjOiHHMjIjs+2qkvQLU=; b=RcHE5uop1Y73pyRCVQkkd2Rw7gcdn72kr5eH/iyKdXNA+w6Zi18WYQUV4ovZtKkLTJ+Mdk2mtutnZ9kbJ5OayWFsuEpyjEWmVsiE6act8m5mUXZZgCW+oIShOCgYDYdHgLzq1ENkRN9nSIBTNxPYfFzwPcWFwpXDgXIe2WDXkFA=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=BQyPzWGIQb6gzaoPAe8KtWE89LoMCv87e4BlqOZNbyfNNVZMWICNApsx/yHJZNBoVt3G0M0W6DM77RKTxqwmizhk0I70dt42urfKc/Cl2D8sSjwglA9DgqBTNsPU2IVdGjr1amxeBKGGirOsInaTZ5v8k6w+uM4ItzYgDOH/dmU=
- References: <200804231933.46291.franke.daniel@gmail.com> <C49D969B-7325-4D58-95F5-658C8C78C4A5@gmail.com> <200805210923.25263.franke.daniel@gmail.com> <4839C0A5.2060404@net-b.de>
On Mon, May 26, 2008 at 7:40 AM, Tobias Burnus <burnus@net-b.de> wrote:
> Daniel Franke wrote:
>>
>> Attached update of the Fortran side of the previous patch not only fixes
>> FX' requests, but also adds yet another option, namely -nocpp. Tobias B.
>> asked for this as a possibility to disable preprocessing if it would
>> otherwise be enabled by the respective file extensions.
>>
>> Btw, the C-side of things still needs reviewing. Condensed patch:
>> http://gcc.gnu.org/ml/fortran/2008-05/msg00248.html
>>
>> Bootstrapped and regression tested on i686-pc-linux-gnu. Ok for trunk?
>>
>
> OK. Thanks for the patch. Can you also update the news section in the wiki?
>
> Tobias
>
This patch also breaks mingw32
../../src/gcc/fortran/cpp.c: In function 'cpp_define_builtins':
../../src/gcc/fortran/cpp.c:222: error: 'flag_iso' undeclared (first
use in this function)
../../src/gcc/fortran/cpp.c:222: error: (Each undeclared identifier is
reported only once
../../src/gcc/fortran/cpp.c:222: error: for each function it appears in.)
../../src/gcc/fortran/cpp.c:222: warning: implicit declaration of
function 'builtin_define_with_int_value'
../../src/gcc/fortran/cpp.c:222: error: 'intmax_type_node' undeclared
(first use in this function)
../../src/gcc/fortran/cpp.c:222: warning: type defaults to 'int' in
declaration of '__t'
../../src/gcc/fortran/cpp.c:222: error: invalid type argument of '->'
(have 'int')
../../src/gcc/fortran/cpp.c:222: warning: passing argument 1 of
'tree_class_check_failed' makes pointer from integer without a cast
../../src/gcc/fortran/cpp.c:222: error: invalid type argument of '->'
(have 'int')
In TARGET_OS_CPP_BUILTINS()
config/i386/cygming.h references 'flag_iso'
config/i386/mingw32.h references 'builtin_define_with_int_value'
and 'intmax_type_node'
Danny