This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ patch, testsuite, fortran] Replace "call abort" by "stop n"
- From: Janus Weil <janus at gcc dot gnu dot org>
- To: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
- Cc: Thomas Koenig <tkoenig at netcologne dot de>, "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: Fri, 16 Feb 2018 09:10:48 +0100
- Subject: Re: [ patch, testsuite, fortran] Replace "call abort" by "stop n"
- Authentication-results: sourceware.org; auth=none
- References: <9fe99d94-89d0-41ba-e298-fb853d8bd02a@netcologne.de> <93359a9c-8b45-c7e4-b6ec-c75a0812a78b@netcologne.de> <20180215233746.GA83215@troutmask.apl.washington.edu>
2018-02-16 0:37 GMT+01:00 Steve Kargl <sgk@troutmask.apl.washington.edu>:
> On Fri, Feb 16, 2018 at 12:25:11AM +0100, Thomas Koenig wrote:
>> I wrote:
>>
>> > OK for trunk?
>>
>> Following a remark by Steve, here is an additional test case which
>> checks if abort actually works. I'll add that to the patch
>> when it is committed.
>>
>
> The patch is ok from my viewpoint. You might want to
> give others a chance to chime in.
Ok with me as well. (Personally I'd have gone with a lowercase "stop",
but uppercase is also fine.)
Thanks for taking care of this! The hardest part was probably the ChangeLog ;)
Cheers,
Janus