This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Structural CFG analysis code


> Jan Hubicka <jh@suse.cz> writes:
> 
> > [...]
> > + /* This pass performs the structural analysis and produces the reduced
> > +    flowgraph containing single entry-multiple exit nodes. 
> > +    It is almost identical to the algorithm described by Munick's
> > +    Advanced Compiler Design and Implementation book.  */
> 
> Please add pages 202-214 ;-)
> > + 
> > + #include "config.h"
> > + #include "system.h"
> > + #include "coretypes.h"
> > + #include "tm.h"
> > + #include "alloc-pool.h"
> > + #include "basic-block.h"
> > + #include "sa.h"
> > + #include "toplev.h"
> > + #include "rtl.h"	/* We need abort() macro.  */
> > + 
> > + /* Pool used to allocate lists of improper regions.  */
> > + alloc_pool list_pool;
> > + /* Dominance info is used to minimize improper regions
> > +    and is computed lazilly when such region appears.  */
> 
> such *a* region
> > + /* Print INDENT spaces.  */
> > + static void
> > + print_indent (FILE * out, int indent)
> 
> I think the coding conventions like to see this as "FILE *out".

Hmm, I write it this way but it is what indent -gnu did to my sources...
I duno :)

I will make the other changes in my local tree.  Thanks!

Honza


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]