From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24317 invoked by alias); 17 Oct 2011 06:02:21 -0000 Received: (qmail 24288 invoked by uid 22791); 17 Oct 2011 06:02:18 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from mail-wy0-f175.google.com (HELO mail-wy0-f175.google.com) (74.125.82.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 17 Oct 2011 06:02:04 +0000 Received: by wyg19 with SMTP id 19so790966wyg.20 for ; Sun, 16 Oct 2011 23:02:03 -0700 (PDT) MIME-Version: 1.0 Received: by 10.216.25.196 with SMTP id z46mr6472675wez.16.1318831322941; Sun, 16 Oct 2011 23:02:02 -0700 (PDT) Received: by 10.216.234.3 with HTTP; Sun, 16 Oct 2011 23:02:02 -0700 (PDT) In-Reply-To: References: Date: Mon, 17 Oct 2011 06:53:00 -0000 Message-ID: Subject: Re: [Patch, Fortran] PR 47023: [4.6/4.7 regression] C_Sizeof: Rejects valid code From: Paul Richard Thomas To: Janus Weil Cc: gfortran , gcc-patches Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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: 2011-10/txt/msg01453.txt.bz2 Dear Janus, Of course you can commit to 4.6. Be quick, though; 4.6.2 was due for release now-ish - "GCC 4.6 branch remains open under normal release branch rules, accepting regression and documentation fixes. GCC 4.6.2 is tentatively planned for late September or early October." Thanks Paul On Sun, Oct 16, 2011 at 9:45 PM, Janus Weil wrote: > Hi Paul, > >> This is OK for trunk. =A0Thanks fo rthe patch. > > Thanks. Committed to trunk as r180062. What about 4.6? > > Cheers, > Janus > > > >> On Sun, Oct 16, 2011 at 2:58 PM, Janus Weil wrote: >>> Hi all, >>> >>> here is a patch which fixes the regression in comment #2 of the PR in >>> the subject line. What it does is setting the 'ts.is_c_interop' flag >>> correctly for constants with kind-parameter specification (such as >>> '0.0_c_double'), as is already being done for variables. >>> >>> Regtested on x86_64-unknown-linux-gnu. Ok for trunk and 4.6? >>> >>> Cheers, >>> Janus >>> >>> >>> 2011-10-16 =A0Janus Weil =A0 >>> >>> =A0 =A0 =A0 =A0PR fortran/47023 >>> =A0 =A0 =A0 =A0* primary.c (match_kind_param): Detect ISO_C_BINDING kin= ds. >>> =A0 =A0 =A0 =A0(get_kind): Pass on 'is_iso_c' flag. >>> =A0 =A0 =A0 =A0(match_integer_constant,match_real_constant,match_logica= l_constant): >>> =A0 =A0 =A0 =A0Set 'ts.is_c_interop'. >>> >>> >>> 2011-10-16 =A0Janus Weil =A0 >>> >>> =A0 =A0 =A0 =A0PR fortran/47023 >>> =A0 =A0 =A0 =A0* gfortran.dg/c_kind_tests_3.f03: New. >>> >> >> >> >> -- >> The knack of flying is learning how to throw yourself at the ground and = miss. >> =A0 =A0 =A0=A0 --Hitchhikers Guide to the Galaxy >> > --=20 The knack of flying is learning how to throw yourself at the ground and mis= s. =A0 =A0 =A0=A0 --Hitchhikers Guide to the Galaxy