This is the mail archive of the
mailing list for the GCC project.
Re: Results for 2.97 20010212 (experimental) testsuite on i686-pc -cyg win
- To: Neil Booth <neil at daikokuya dot demon dot co dot uk>
- Subject: Re: Results for 2.97 20010212 (experimental) testsuite on i686-pc -cyg win
- From: Christopher Faylor <cgf at redhat dot com>
- Date: Wed, 28 Feb 2001 19:51:31 -0500
- Cc: dj at redhat dot com, "Billinghurst, David (CRTS)" <David dot Billinghurst at riotinto dot com>, gcc-bugs at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- References: <8D00C32549556B4E977F81DBC24E985DC6E2@crtsmail1.technol_exch.corp.riotinto.org> <20010228074833.C28670@daikokuya.demon.co.uk>
On Wed, Feb 28, 2001 at 07:48:33AM +0000, Neil Booth wrote:
>Does this look OK to you? If so, OK to commit for branch and mainline?
I've been working in this section of the code. I missed the earlier
discussion about this -- I guess I should start reading gcc-bugs, maybe?
Anyway, I don't know what "this problem" is. Could you elucidate?
The intent of the code you are changing was to duplicate the normal search
order that I thought gcc used. I thought the -isystem ... -idirafter
would cause /usr/local/include to be searched before any target specific
includes and /usr/include to be searched after the target specific includes.
If it is incorrect I'd like to know why.
>Billinghurst, David (CRTS) wrote:-
>> I have a solution to this problem. It may even be correct.
>> The cause of the problem is using "-idirafter /usr/include" rather than
>> "-isystem /usr/include". This is fixed with the following patch, which
>> bootstraps on the gcc-3.0 branch and eliminates the cpp testsuite failures.
>> --- gcc/config/i386/cygwin.h.orig Tue Feb 6 21:11:08 2001
>> +++ gcc/config/i386/cygwin.h Tue Feb 27 20:08:33 2001
>> @@ -88,7 +88,7 @@
>> #define MINGW_LIBS "-L" CYGWIN_CROSS_DIR "/lib/mingw"
>> #define MINGW_INCLUDES "-I" CYGWIN_CROSS_DIR "/include/mingw"
>> -#define CYGWIN_INCLUDES "-isystem /usr/local/include -idirafter
>> +#define CYGWIN_INCLUDES "-isystem /usr/local/include -isystem /usr/include"
>> #define CYGWIN_W32API "-I/usr/include/w32api"
>> #define CYGWIN_LIB "/usr/lib"
>> #define MINGW_LIBS "-L/usr/local/lib/mingw -L/usr/lib/mingw"
>> I don't understand why -idirafter was used. There are only a few
>> occurrences of it in the source tree. If it wrong for cygwin it may be wrong
>> elsewhere as well.
>> Your thoughts?