From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-patches-return-201168-listarch-gcc-patches=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 20199 invoked by alias); 9 Aug 2007 21:31:25 -0000
Received: (qmail 20161 invoked by uid 22791); 9 Aug 2007 21:31:23 -0000
X-Spam-Check-By: sourceware.org
Received: from mail.artimi.com (HELO mail.artimi.com) (194.72.81.2)     by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 09 Aug 2007 21:31:16 +0000
Received: from rainbow ([192.168.8.46]) by mail.artimi.com with Microsoft SMTPSVC(6.0.3790.3959); 	 Thu, 9 Aug 2007 22:30:55 +0100
From: "Dave Korn" <dave.korn@artimi.com>
To: "'FX Coudert'" <fxcoudert@gmail.com>, 	"'Tobias Burnus'" <burnus@net-b.de>
Cc: <fortran@gcc.gnu.org>, 	"'gcc-patches'" <gcc-patches@gcc.gnu.org>
References: <46B7657F.2070204@net-b.de> <D620FA1F-4BF2-4298-9978-52BA0EF5F2D6@gmail.com>
Subject: RE: [Fortran, patch] PR33001 - Point to -fno-range-check
Date: Thu, 09 Aug 2007 21:31:00 -0000
Message-ID: <033c01c7dacc$9191bf20$2e08a8c0@CAM.ARTIMI.COM>
MIME-Version: 1.0
Content-Type: text/plain; 	charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Office Outlook 11
In-Reply-To: <D620FA1F-4BF2-4298-9978-52BA0EF5F2D6@gmail.com>
X-IsSubscribed: yes
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-patches.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-help@gcc.gnu.org>
Sender: gcc-patches-owner@gcc.gnu.org
X-SW-Source: 2007-08/txt/msg00602.txt.bz2

On 09 August 2007 22:15, FX Coudert wrote:

>> REVIEWMAIL:
> 
>> Index: gcc/fortran/arith.c
>> ===================================================================
>> --- gcc/fortran/arith.c (Revision 127246)
>> +++ gcc/fortran/arith.c (Arbeitskopie)
>> @@ -1981,7 +1981,8 @@ arith_error (arith rc, gfc_typespec *fro
>>                  gfc_typename (from), gfc_typename (to), where);       
>>      break; case ARITH_OVERFLOW:
>> -      gfc_error ("Arithmetic overflow converting %s to %s at %L",
>> +      gfc_error ("Arithmetic overflow converting %s to %s at %L. This
>> check " +                "can be disabled with the option
>>                  -fno-range-check", gfc_typename (from), gfc_typename
>>      (to), where);        break; case ARITH_UNDERFLOW:
> 
> OK, noone seems to have any objection and I find it nice.
> 
> FX

  Dunno if you guys are interested in a random suggestion but it might be even
nicer if the second sentence was only printed the first time per run.


    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....