This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, fortran] Fix PR 54033, problems with -I, with test cases
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Tobias Burnus <burnus at net-b dot de>
- 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: Thu, 2 Aug 2012 10:54:04 +0200
- Subject: Re: [patch, fortran] Fix PR 54033, problems with -I, with test cases
- References: <50117B5D.8030304@netcologne.de> <50118A67.2040702@mentor.com> <5012F4AF.7050807@netcologne.de> <50130F9B.1010300@mentor.com> <5015015F.6070303@netcologne.de> <5017E1F6.3050906@net-b.de>
On Tue, Jul 31, 2012 at 3:47 PM, Tobias Burnus <burnus@net-b.de> wrote:
> On 07/29/2012 11:24 AM, Thomas Koenig wrote:
>>
>> here is an updated patch for PR 54033, this time with test cases.
>> Thanks to Janis for pointing me in the right direction with these.
>> Regression-tested. OK for trunk?
>
>
> Ok. Thanks for the patch - and to Janis for the first review.
>
> Can you eliminate the extra line before "else if"?
Seems to break testing, all testcases emit
Warning: Nonexistent include directory "finclude"^M
now and thus all testcases fail like
FAIL: gfortran.dg/alloc_comp_basics_1.f90 -O0 (test for excess errors)
Richard.
> Tobias
>
>
>> 2012-07-29 Thomas König <tkoenig@gcc.gnu.org>
>>
>> PR fortran/54033
>> * scanner.c (add_path_to_list): Emit warning if an error occurs
>> for an include path, if it is not present or if it is not a
>> directory. Do not add the path in these cases.
>>
>> 2012-07-29 Thomas König <tkoenig@gcc.gnu.org>
>>
>> PR fortran/54033
>> * gfortran.dg/include_6.f90: New test case.
>> * gfortran.dg/include_7.f90: New test case.
>> * gfortran.dg/include_3.f90: Add dg-warning for missing
>> directory.
>>
>>
>