[PATCH]: Add RHS to V_MUST_DEF

Daniel Berlin dberlin@dberlin.org
Tue Oct 26 16:33:00 GMT 2004


>> +      (mark_def_block_sites).
>>         ^^^^^^^^^^^^^^^^^^^^
>          mark_def_site_blocks

Fixed.

>>
> I'd move htab_delete (def_blocks) inside rewrite_blocks.  You'll also
> need to remove it from rewrite_into_ssa.

Will do.

>
>> @@ -420,6 +420,11 @@ execute_todo (int properties, unsigned i
>>         rewrite_into_ssa (false);
>>         bitmap_clear (vars_to_rename);
>>       }
>> +  if (flags & TODO_fix_def_def_chains)
>> +    {
>> +      rewrite_def_def_chains ();
>> +      bitmap_clear (vars_to_rename);
>> +    }
>>
> Silly optimization, but if TODO_rename_vars is also set, we don't need
> to call rewrite_def_def_chains.

True enough.
I'll make it an "else if"

>
>
> OK with those changes.
>
>
> Diego.
>
>



More information about the Gcc-patches mailing list