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 for fortran/16290

From: Richard Sandiford <rsandifo at redhat dot com>
To: Steven Bosscher <stevenb at suse dot de>
Cc: "Billinghurst, David (CALCRTS)"<david dot billinghurst at comalco dot riotinto dot com dot au>, <gcc-patches at gcc dot gnu dot org>, <fortran at gcc dot gnu dot org>
Date: Wed, 30 Jun 2004 09:34:07 +0100
Subject: Re: Patch for fortran/16290
References: <8ED2FA87C8ACC04192709688DDFD5F5F4362D8@calttsv025.cal.riotinto.org><200406301016.15287.stevenb@suse.de>

Steven Bosscher <stevenb@suse.de> writes:
> On Wednesday 30 June 2004 06:57, Billinghurst, David (CALCRTS) wrote:
>> Test intrinsic_set_exponent.f90 fails on irix6.5 as it is
>> using exponents smaller than minexponent(x).  Fixed thus.
>>
>> OK to commit?
>>
>> 2004-06-30  David Billinghurst  (David.Billinghurst@riotinto.com)
>>
>> 	PR fortran/16290
>> 	* gfortran.fortran-torture/execute/intrinsic_set_exponent.f90
>> 	Don't use values of n smaller than minexponent(x).
>
> I'd suggest you xfail it instead.

But it looks like David's patch makes the test more portable.
Surely that's a good thing?

There are other tests in the same file that should pass on IRIX,
and which David didn't need to change.  It'd be nice not to have
to skip them for the sake of the ones that don't pass.

Richard

Follow-Ups:
- Re: Patch for fortran/16290
  - From: Steven Bosscher

References:
- Patch for fortran/16290
  - From: Billinghurst, David (CALCRTS)
- Re: Patch for fortran/16290
  - From: Steven Bosscher

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