[PATCH]: Interprocedural detection of readonly and non-addressable static variables

Gabriel Dos Reis gdr@integrable-solutions.net
Wed Jun 8 15:28:00 GMT 2005


Daniel Berlin <dberlin@dberlin.org> writes:

[...]

| > | 2005-06-06  Kenneth Zadeck  <zadeck@naturalbridge.com>
| > |        Danny Berlin <dberlin@dberlin.org>
| > | 
| > |    Interprocedual detection of readonly and non-addressable static
| > |    variables.
| > | 
| > |    * ipa-reference.c, ipa-reference.h, ipa-utils.c, ipa-utils.h: new
| > |    files.
| > |    * common.opt: Added "fipa-reference".
| > |    * opts.c: Added "flag_ipa_reference".
| > |    * timevar.def: Added TV_IPA_REFERENCE.
| > |    * tree-dfa.c (find_reference_vars) Addition of call to reset
| > |    per function information.
| > |    * tree-flow.h: Addition of reference_vars_info field for
| > |    FUNCTION_DECLS.
| > |    * tree-optimize.c: Added pass.
| > |    * tree-pass.h: Added pass.
| > | 
| > | 
| > | 
| > | Bootstrapped and regtested on i686-pc-linux-gnu and powerpc-linux-gnu.
| > | 
| > | Okay for mainline?
| > 
| > I think it makes sense to put the core files in a seperate directory,
| > say gcc/ipa.  That helps strenghting the GCC structure. We have too
| > much of otherwise reasonably partitioned component files lumped
| > together in gcc/. 
| 
| I'm not sure how to pull that off, however.
| 
| Is it as simple as prefixing the files with ipa/ in the Makefile?

That should be a good enough start.

| Or did you want me to add a new makefile to that dir?

well, that would be nice but I'm not sure our build machinery is
"simple" enough to make that easy.  I would be happy with your first
suggestion but if you think you can add more, well, I'll take it! :-)

-- Gaby



More information about the Gcc-patches mailing list