Re: [PATCH] Fix PR56321

2013-02-18 Thread Richard Biener
On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt wschm...@linux.vnet.ibm.com 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

[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 rguent...@suse.de PR tree-optimization/56321 * tree-ssa-reassoc.c (propagate_op_to_single_use): Properly order

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 wschm...@linux.vnet.ibm.com 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

[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