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: [PATCH] PR other/63613: Add fixincludes for dejagnu.h


On 12/04/14 15:19, David Malcolm wrote:
<dejagnu.h> assumed -fgnu89-inline until a recent upstream fix;
see http://lists.gnu.org/archive/html/dejagnu/2014-10/msg00011.html

Remove the workaround from jit.exp that used -fgnu89-inline
in favor of a fixincludes to dejagnu.h that applies the upstream fix
to a local copy.

This should make it easier to support C++ testcases from jit.exp.

(I also needed to fix up the jit.dg/test-threads.c due to the
preprocessor tricks that that test plays in order to make
dejagnu.h be threadsafe).

This is the first time I've touched the "fixincludes" directory;
is this the correct way to make a change here?

Successfully bootstrapped & regrtested on x86_64-unknown-linux-gnu
(Fedora 20).

OK for trunk?

fixincludes/ChangeLog:
	PR other/63613
	* inclhack.def (dejagnu_h_make_inline_functions_static): New fix.
	* fixincl.x: Regenerate.
	* tests/base/dejagnu.h: New.
OK.
jeff


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