On Mon, 2005-08-01 at 17:27 +0300, Dorit Naishlos wrote: > > > > > ... > > > > The problem seems to be that analyze_offset_expr calls the scev > > analyzer explicitely asking for recomputation (third parameter is > > true): > > > > ... > > > > Why should we start the analysis from scratch in this case? The same > > question could be asked for all the uses of analyze_scalar_evolution > > in the vectorizer... Grepping for analyze_scalar_evolution, you get > > the following places (in autovect branch) that explicitely ask for > > scev recomputation: > > > > I don't think there was an intention to force recomputation - probably just > overlooked what the third argument actually stands for. These occurrences > could probably be changed to false. >
Yeah, i agree with dorit. The only other possible reason i can think of is that it fixed a bug somewhere due to not properly invalidating the cache, and that fix was just copied around. > dorit >