Re: [PATCH] PR other/63613: Add fixincludes for dejagnu.h

[Jeff Law]

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