public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [plugin] Add output.h and ipa-utils.h to PLUGIN_HEADERS
@ 2009-10-22 18:31 Diego Novillo
  2009-10-22 19:25 ` Paolo Bonzini
  0 siblings, 1 reply; 3+ messages in thread
From: Diego Novillo @ 2009-10-22 18:31 UTC (permalink / raw)
  To: gcc-patches, tglek

For a new dehydra feature
(https://bugzilla.mozilla.org/show_bug.cgi?id=523435), I need to
use output.h and ipa-utils.h from plugin code.

I'm not completely sure if this is allowed during stage3.  It
seems innocent enough, though.

Tested on x86_64 via make install.


Thanks.  Diego.


	* Makefile.in (PLUGIN_HEADERS): Add output.h and
	IPA_UTILS_H.

Index: Makefile.in
===================================================================
--- Makefile.in	(revision 153467)
+++ Makefile.in	(working copy)
@@ -4260,7 +4260,7 @@ PLUGIN_HEADERS = $(TREE_H) $(CONFIG_H) $
   $(host_xm_file_list) $(host_xm_include_list) $(xm_include_list) \
   intl.h $(PLUGIN_VERSION_H) $(DIAGNOSTIC_H) $(C_COMMON_H) $(C_PRETTY_PRINT_H) \
   tree-iterator.h $(PLUGIN_H) $(TREE_FLOW_H) langhooks.h incpath.h \
-  tree-ssa-sccvn.h real.h
+  tree-ssa-sccvn.h real.h output.h $(IPA_UTILS_H)
 
 # Install the headers needed to build a plugin.
 install-plugin: installdirs lang.install-plugin

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

* Re: [plugin] Add output.h and ipa-utils.h to PLUGIN_HEADERS
  2009-10-22 18:31 [plugin] Add output.h and ipa-utils.h to PLUGIN_HEADERS Diego Novillo
@ 2009-10-22 19:25 ` Paolo Bonzini
  2009-10-22 20:24   ` Diego Novillo
  0 siblings, 1 reply; 3+ messages in thread
From: Paolo Bonzini @ 2009-10-22 19:25 UTC (permalink / raw)
  To: Diego Novillo; +Cc: gcc-patches, tglek

On 10/22/2009 08:06 PM, Diego Novillo wrote:
> For a new dehydra feature
> (https://bugzilla.mozilla.org/show_bug.cgi?id=523435), I need to
> use output.h and ipa-utils.h from plugin code.
>
> I'm not completely sure if this is allowed during stage3.  It
> seems innocent enough, though.

I don't see why not (and with the LTO merge still settling, it's quite a 
permissive stage3).

Paolo

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

* Re: [plugin] Add output.h and ipa-utils.h to PLUGIN_HEADERS
  2009-10-22 19:25 ` Paolo Bonzini
@ 2009-10-22 20:24   ` Diego Novillo
  0 siblings, 0 replies; 3+ messages in thread
From: Diego Novillo @ 2009-10-22 20:24 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gcc-patches, tglek

On Thu, Oct 22, 2009 at 15:22, Paolo Bonzini <bonzini@gnu.org> wrote:

> I don't see why not (and with the LTO merge still settling, it's quite a
> permissive stage3).

Thanks.  We discussed it on IRC and agreed that it should not be a
problem, it affects a new directory and a new feature only, so it
should not cause regressions.


Diego.

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

end of thread, other threads:[~2009-10-22 19:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-10-22 18:31 [plugin] Add output.h and ipa-utils.h to PLUGIN_HEADERS Diego Novillo
2009-10-22 19:25 ` Paolo Bonzini
2009-10-22 20:24   ` Diego Novillo

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