This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFA: Testsuite: Prevent limits.h from using #include_next instrict ANSI mode


Hi Joseph,


 The problem was that a GCC version of the system header file
 <limits.h> was being included, and this header used the
 #include_next <> preprocessor operation to chain on to the next


As I said before <http://gcc.gnu.org/ml/gcc/2003-05/msg01670.html>, the system headers provided by GCC should be included by the testsuite harness with -isystem, please fix the bug there instead. The tests are correct; it is correct to include <limits.h> in strict ANSI mode and not worry about its internals. The problems was tracked to DejaGnu in the previous thread; and indeed it is fixed in DejaGnu 1.4.4 (documented in install.texi as the minimum required version), probably by


2002-11-12 Hans-Peter Nilsson <hp@bitrange.com>

        * lib/libgloss.exp (newlib_include_flags): Use -isystem, not -I.
        (libio_include_flags, g++_include_flags, libstdc++_include_flags,
        winsup_include_flags): Ditto.

Thanks very much for pointing this out to me. I was unaware of this patch, but now that I have checked I found that it is the cure needed. I withdraw my patch and I will update the dejagnu sources in binutils to include this fix.


Cheers
  Nick



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]