[patch,libfortran] [4.4 Regression] PR38430 gfortran.dg/streamio_1.f90, 10, 14, 2, 6 now fails

Jerry DeLisle jvdelisle@verizon.net
Tue Dec 9 03:24:00 GMT 2008


This patch fixes the regression on CRIS/ELF. It was really broken on all 
platforms but did not show up in testsuite. The error was caused by using 
dtp->rec instead of dtp->pos while checking that the position did not exceed the 
maximum value allowed.

I also commented out a constraint that would cause 4.3 executables to fail with 
4.4 library.  This code will be uncommented later when we have to break ABI for 
other reasons.  The effect of not having the check is to accept REC= with STREAM 
access. The value passed to REC= will be treated as if it is POS=.

I will commit for now and if anyone objects to this solution I can simply 
un-comment it.

Regression tested on x86-64.

Regards,

Jerry

2008-12-08  Jerry DeLisle  <jvdelisle@gcc.gnu.org>

	PR libfortran/38430
	* io/transfer.c (data_transfer_init): Move constraint check for REC=
	and STREAM access into condition for other REC= constraints to simplify.
	Comment out this constraint to avoid breaking compatibility with 4.3
	until later.  Added a TODO for above comment and this one.
	Fix test for max pos.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pr38430.diff
Type: text/x-diff
Size: 1571 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20081209/61f11ac9/attachment.bin>


More information about the Gcc-patches mailing list