Re: [PATCH] Fix PR56321

2013-02-18 Thread Bill Schmidt
On Mon, 2013-02-18 at 13:05 +0100, Richard Biener wrote: > On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt > wrote: > > When we remove __builtin_pow statements as part of reassociation, we > > have to unlink the associated VDEF. We've always done this when we > > directly remove the statement. How

[PATCH] Fix PR56321

2013-02-18 Thread Richard Biener
This fixes PR56321 - the unlink_stmt_vdef was a no-op as it was called after release_defs. Oops. Committed as obvoious. Richard. 2013-02-18 Richard Biener PR tree-optimization/56321 * tree-ssa-reassoc.c (propagate_op_to_single_use): Properly order SSA name release a

Re: [PATCH] Fix PR56321

2013-02-18 Thread Richard Biener
On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt wrote: > When we remove __builtin_pow statements as part of reassociation, we > have to unlink the associated VDEF. We've always done this when we > directly remove the statement. However, in reassociation the statements > are sometimes modified in p

[PATCH] Fix PR56321

2013-02-15 Thread Bill Schmidt
When we remove __builtin_pow statements as part of reassociation, we have to unlink the associated VDEF. We've always done this when we directly remove the statement. However, in reassociation the statements are sometimes modified in place instead of removed, potentially leaving one or more dangl