On 11/12/12, Diego Novillo <dnovi...@google.com> wrote:
> On Mon, Nov 12, 2012 at 2:31 PM, Lawrence Crowl <cr...@googlers.com> wrote:
>> On 11/12/12, Lawrence Crowl <cr...@googlers.com> wrote:
>>> It appears that
>>>
>>>   static bitmap clear_alias_sets = NULL;
>>>
>>> is never set, and as a consequence
>>>
>>>   clear_alias_set_lookup (alias_set_type alias_set)
>>>
>>> is never called.  This conclusion is reinforced because it
>>> unconditionally uses htab_t clear_alias_mode_table, which is never
>>> allocated.
>>
>> This is in dse.c.
>
> I looked at the comment above clear_alias_sets and it states that it
> is only set by dse_record_singleton_alias_set, which has disappeared
> from the source code.
>
> Richi, you removed dse_record_singleton_alias_set in March.  Perhaps
> this whole thing is dead code and should be removed?

Richi, ping?

-- 
Lawrence Crowl

Reply via email to