This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [Patch, fortran] Fix PR53876 (regression), PR55990 (regression) and PR55992

From: Tobias Burnus <burnus at net-b dot de>
To: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Sun, 06 Jan 2013 15:25:44 +0100
Subject: Re: [Patch, fortran] Fix PR53876 (regression), PR55990 (regression) and PR55992
References: <CAGkQGiKgb_Fbk+QxHexh8wMJK=XzTCsUBWjJ=QMH1_vBV5KX5A@mail.gmail.com>

Paul Richard Thomas wrote:

Boostratpped and regtested on FC17/x86_64 - OK for trunk?

OK - with the below nits fixed. Thanks for the patch.

     PR fortran/PR53876
     PR fortran/PR55990
     PR fortran/PR55992

Change 55* into 54*.

     PR fortran/PR53876
     PR fortran/PR55990
     PR fortran/PR55992

Ditto.

+ /* Ensure that the CLASS langauge specific flag is set. */

s/langauge/language/

+ ! { dg-do run}

s/run}/run }/

Without the " " before the right curly brace, DejaGNU will ignore the line and handle it as "dg-do compile". Please check after that change that the test case doesn't fail at run time.

Tobias

Follow-Ups:
- Re: [Patch, fortran] Fix PR53876 (regression), PR55990 (regression) and PR55992
  - From: Paul Richard Thomas

References:
- [Patch, fortran] Fix PR53876 (regression), PR55990 (regression) and PR55992
  - From: Paul Richard Thomas

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]