Re: [patch] Remove redundant calls to copy_virtual_operands

[Zdenek Dvorak <rakdver@kam.mff.cuni.cz>]

Hello,

> On 8/7/07 9:07 AM, Dorit Nuzman wrote:
> 
> > it looks ok to me, but it would be much better if one of the aliasing/vops
> > experts took a look
> 
> Yeah, it is OK.  As I mentioned earlier, I would only suggest doing
> compile-time tests.  Though if this fixes ICEs, then it's really better
> even if a tad slower.  Leaving this to update_ssa is certainly easier to
> maintain.

on the other hand, the fact that there are ICEs shows that there is some
problem in setting the alias information for the new statements in
vectorizer -- if this were correct, the virtual operands would be
exactly the same as the ones of the original statement; so although I
agree that leaving things to update_ssa is easier to maintain (although
possibly much slower), it would be nice if someone could check what does
the alias information updating code in vectorizer get wrong,

Zdenek