PR tree-optimization/54985

Jeff Law law@redhat.com
Tue Oct 23 21:28:00 GMT 2012


On 10/23/2012 02:50 PM, Jakub Jelinek wrote:
> On Tue, Oct 23, 2012 at 02:35:24PM -0600, Jeff Law wrote:
>> +/* Return TRUE if the statement at the end of e->dest depends on
>> +   the output of any statement in BB.   Otherwise return FALSE.
>> +
>> +   This is used when we are threading a backedge and need to ensure
>> +   that temporary equivalences from BB do not affect the condition
>> +   in e->dest.  */
>> +
>> +static bool
>> +cond_arg_set_in_bb (edge e, basic_block bb, int n)
>> +{
>> +  ssa_op_iter iter;
>> +  use_operand_p use_p;
>> +  gimple last = gsi_stmt (gsi_last_bb (e->dest));
>
> Use gimple last = last_stmt (e->dest); instead?  That way any possible
> debug stmts are ignored.
I thought I'd already dealt with this before.  I'll double-check and 
take appropriate action.

Any opinions about pulling it into 4.7.x as that release is affected by 
this codegen bug?  I've got no strong opinions and I'm willing to pull 
it onto the branch if you want it.

jeff



More information about the Gcc-patches mailing list