This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: PR22336 (was: Problem with tree-ssa-loop-ivopts.c:get_computation-cost)
- From: Zdenek Dvorak <rakdver at atrey dot karlin dot mff dot cuni dot cz>
- To: Richard Kenner <kenner at vlsi1 dot ultra dot nyu dot edu>
- Cc: gcc-patches at gcc dot gnu dot org, gcc at gcc dot gnu dot org
- Date: Tue, 12 Jul 2005 15:55:26 +0200
- Subject: Re: PR22336 (was: Problem with tree-ssa-loop-ivopts.c:get_computation-cost)
- References: <10507121349.AA07335@vlsi1.ultra.nyu.edu>
Hello,
> I think the patch is fine (although of course I cannot approve it).
>
> I, as it's author, do not. But, as I keep saying, I don't know what
> the proper fix is.
preventing record_block_change from working in this situation seems a
quite clean solution to me; of course, not expanding expressions with
TREE_BLOCK set (when determining their cost) at all would be better, but
somewhat more complicated.
> The more proper fix would be to never expand such expressions from ivopts;
>
> What are "such expressions"?
Expressions with TREE_BLOCK set; i.e. those coming from the compiled program.
Zdenek