public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* [trunk] hard-reg-set.h not installed in plugins/include
@ 2010-10-16 21:15 Basile Starynkevitch
  2010-10-16 21:29 ` Basile Starynkevitch
  0 siblings, 1 reply; 3+ messages in thread
From: Basile Starynkevitch @ 2010-10-16 21:15 UTC (permalink / raw)
  To: gcc


Hello,

It seems that the file hard-reg-set.h is included from gimple.h
but it is not installed in the PLUGIN_HEADERS (inside gcc/Makefile.in)

So I believe that any plugin for gcc trunk (at least rev 165392) which
#include-s "gimple.h" cannot be compiled.

(I actually noticed that bug in the MELT branch, 
but it follows the trunk in that respect).

Did anyone installed the trunk recently and compiled a plugin including
"gimple.h" for it?

Cheers.
-- 
Basile STARYNKEVITCH         http://starynkevitch.net/Basile/
email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
8, rue de la Faiencerie, 92340 Bourg La Reine, France
*** opinions {are only mine, sont seulement les miennes} ***

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [trunk] hard-reg-set.h not installed in plugins/include
  2010-10-16 21:15 [trunk] hard-reg-set.h not installed in plugins/include Basile Starynkevitch
@ 2010-10-16 21:29 ` Basile Starynkevitch
  2010-10-16 22:15   ` Basile Starynkevitch
  0 siblings, 1 reply; 3+ messages in thread
From: Basile Starynkevitch @ 2010-10-16 21:29 UTC (permalink / raw)
  To: Basile Starynkevitch; +Cc: gcc

On Sat, 16 Oct 2010 23:14:18 +0200
Basile Starynkevitch <basile@starynkevitch.net> wrote:

> 
> Hello,
> 
> It seems that the file hard-reg-set.h is included from gimple.h
> but it is not installed in the PLUGIN_HEADERS (inside gcc/Makefile.in)
> 
> So I believe that any plugin for gcc trunk (at least rev 165392) which
> #include-s "gimple.h" cannot be compiled.

I believe that $(REGSET_H) should be added to $(PLUGIN_HEADERS) near
line 4506 of gcc/Makefile.in [but I don't propose a patch now, because
I really would prefer my
http://gcc.gnu.org/ml/gcc-patches/2010-10/msg01262.html patch to be
reviewed and Ok-ed first.]

Cheers.

-- 
Basile STARYNKEVITCH         http://starynkevitch.net/Basile/
email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
8, rue de la Faiencerie, 92340 Bourg La Reine, France
*** opinions {are only mine, sont seulement les miennes} ***

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [trunk] hard-reg-set.h not installed in plugins/include
  2010-10-16 21:29 ` Basile Starynkevitch
@ 2010-10-16 22:15   ` Basile Starynkevitch
  0 siblings, 0 replies; 3+ messages in thread
From: Basile Starynkevitch @ 2010-10-16 22:15 UTC (permalink / raw)
  To: Basile Starynkevitch; +Cc: gcc

On Sat, 16 Oct 2010 23:28:37 +0200
Basile Starynkevitch <basile@starynkevitch.net> wrote:
> 
> I believe that $(REGSET_H) should be added to $(PLUGIN_HEADERS) near
> line 4506 of gcc/Makefile.in [but I don't propose a patch now, because
> I really would prefer my
> http://gcc.gnu.org/ml/gcc-patches/2010-10/msg01262.html patch to be
> reviewed and Ok-ed first.]


I just commited such a patch to the MELT branch (which I merged to
trunk rev.165564), I built and installed it, and the installed MELT
works again. So I believe the fix should work even on the trunk.


-- 
Basile STARYNKEVITCH         http://starynkevitch.net/Basile/
email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
8, rue de la Faiencerie, 92340 Bourg La Reine, France
*** opinions {are only mine, sont seulement les miennes} ***

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-10-16 22:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-16 21:15 [trunk] hard-reg-set.h not installed in plugins/include Basile Starynkevitch
2010-10-16 21:29 ` Basile Starynkevitch
2010-10-16 22:15   ` Basile Starynkevitch

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).