public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug plugins/45346]  New: hard-reg-set.h needs to be in the plugin include directory
@ 2010-08-19 18:24 sje at cup dot hp dot com
  2010-08-19 18:48 ` [Bug plugins/45346] " rguenth at gcc dot gnu dot org
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: sje at cup dot hp dot com @ 2010-08-19 18:24 UTC (permalink / raw)
  To: gcc-bugs

I am trying to build a plugin that uses cgraph.h, which is in the plugin
directory.  But cgraph.h needs function.h and function.h needs hard-reg-set.h.

hard-reg-set.h is not in the plugin include directory.

The test case is to build a plugin where the plugin uses the header file
cgraph.h
or function.h.

I believe this started failing at r159776 when Paul Brook added the include of
hard-reg-set.h into function.h


-- 
           Summary: hard-reg-set.h needs to be in the plugin include
                    directory
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: plugins
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: sje at cup dot hp dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45346


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Bug plugins/45346] hard-reg-set.h needs to be in the plugin include directory
  2010-08-19 18:24 [Bug plugins/45346] New: hard-reg-set.h needs to be in the plugin include directory sje at cup dot hp dot com
@ 2010-08-19 18:48 ` rguenth at gcc dot gnu dot org
  2010-08-19 20:25 ` steven at gcc dot gnu dot org
  2010-08-19 20:28 ` steven at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2010-08-19 18:48 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from rguenth at gcc dot gnu dot org  2010-08-19 18:48 -------
Patch to add it is pre-approved.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45346


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Bug plugins/45346] hard-reg-set.h needs to be in the plugin include directory
  2010-08-19 18:24 [Bug plugins/45346] New: hard-reg-set.h needs to be in the plugin include directory sje at cup dot hp dot com
  2010-08-19 18:48 ` [Bug plugins/45346] " rguenth at gcc dot gnu dot org
@ 2010-08-19 20:25 ` steven at gcc dot gnu dot org
  2010-08-19 20:28 ` steven at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: steven at gcc dot gnu dot org @ 2010-08-19 20:25 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from steven at gcc dot gnu dot org  2010-08-19 20:24 -------
Gross.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45346


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Bug plugins/45346] hard-reg-set.h needs to be in the plugin include directory
  2010-08-19 18:24 [Bug plugins/45346] New: hard-reg-set.h needs to be in the plugin include directory sje at cup dot hp dot com
  2010-08-19 18:48 ` [Bug plugins/45346] " rguenth at gcc dot gnu dot org
  2010-08-19 20:25 ` steven at gcc dot gnu dot org
@ 2010-08-19 20:28 ` steven at gcc dot gnu dot org
  2 siblings, 0 replies; 4+ messages in thread
From: steven at gcc dot gnu dot org @ 2010-08-19 20:28 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from steven at gcc dot gnu dot org  2010-08-19 20:28 -------
The proper fix for this is not to just go back to the "include everything
everywhere". The proper fix is instead to move struct rtl_data out of
function.h (as I have said before) and into e.g. emit-rtl.h.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45346


^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-08-19 20:28 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-19 18:24 [Bug plugins/45346] New: hard-reg-set.h needs to be in the plugin include directory sje at cup dot hp dot com
2010-08-19 18:48 ` [Bug plugins/45346] " rguenth at gcc dot gnu dot org
2010-08-19 20:25 ` steven at gcc dot gnu dot org
2010-08-19 20:28 ` steven at gcc dot gnu dot org

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).