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: [tree-ssa]: Add strength reduction to SSAPRE


On Tuesday, September 17, 2002, at 06:10  PM, Diego Novillo wrote:

On Tue, 17 Sep 2002, Daniel Berlin wrote:

	* tree-flow.h: Remove EXPR* macros in favor of inline functions.
	Remove E_INJ.
	
Looks OK, except for some missing newlines to separate the inline
function bodies.

Should I comment these?
I didn't want to because they seem simple enough, and it would double the space they take up.

Maybe we should move these things to tree-flow-inline.h or something, and just #include it?
They seem to dirty up tree-flow.h.
The declarations should still be there, of course, i'm only suggesting moving the implementation.


	* tree-dfa.c: Change EXPR* macros to the new functions.
	Remove E_INJ.

OK.  I was going to point out that you should shift the values
for the other bitmasks now that you removed E_INJ.  But I will be
changing that into an enum shortly, so don't bother.

Bootstrapped and tested, I assume?
Yup.
Both on i686-pc-linux-gnu and powerpc-apple-darwin6.0

Thanks.  Diego.






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