[patch, fortran] PR43851 Add _gfortran_error_stop_numeric
Jerry DeLisle
jvdelisle@verizon.net
Thu May 20 05:31:00 GMT 2010
On 05/19/2010 05:07 AM, Janne Blomqvist wrote:
> On Wed, May 19, 2010 at 04:33, Jerry DeLisle<jvdelisle@verizon.net> wrote:
>> Hi all,
>>
>> I hope this is the final on this patch. This patch avoids breaking ABI by
>> creating two new functions stop_numeric_f2008 and pause_numeric_f2008 along
>> with the error_stop_numeric. This allows the STOP and PAUSE statements to
>> accept constant expressions and not use a value of -1 to signal an empty
>> statement. Existing stop_numeric and pause_numeric are untouched.
>
> Yes, this avoids breaking the ABI. Although I think the same could
> have been achieved by translating an empty stop code to
> (error_)stop_string with a NULL string argument, without having to
> create new library functions. But, this will get the job done as well,
> so I think it's ok, no need to bikeshed it to death.
I liked this idea better, so this is what I did. No duplicate functions created
and no ABI breakage. Added check for default character kind as well. I will
commit a new test case or two next window of opportunity. Seems to work fine
with -fdefault-integer-8. Regression tested on x86-64.
Committed:
$ svn commit
Sending gcc/fortran/ChangeLog
Sending gcc/fortran/match.c
Sending gcc/fortran/trans-decl.c
Sending gcc/fortran/trans-stmt.c
Sending gcc/fortran/trans.h
Sending gcc/testsuite/ChangeLog
Sending gcc/testsuite/gfortran.dg/label_1.f90
Sending libgfortran/ChangeLog
Sending libgfortran/gfortran.map
Sending libgfortran/libgfortran.h
Sending libgfortran/runtime/pause.c
Sending libgfortran/runtime/stop.c
Transmitting file data ............
Committed revision 159609.
Regards,
Jerry
More information about the Gcc-patches
mailing list