On 03/01/2013 11:59 PM, Pedro Alves wrote:
> "improve" is a "Why?", and subjective.  ChangeLogs are objective, and document
> the "What" changed.  I suggest:
>
>           * tilegx-tdep.c (tilegx_analyze_prologue): Limit bundle reading to END_ADDR.
>           (tilegx_skip_prologue): Limit prologue analysis to section end.
>
>>   tilegx_skip_prologue (struct gdbarch *gdbarch, CORE_ADDR start_pc)
>>   {
>> -  CORE_ADDR func_start;
>> +  CORE_ADDR func_start, end_pc;
>> +  struct obj_section *s = NULL;
> No need to initialize 's' here, it's unconditionally initialized below.
>
>> +  s = find_pc_section (start_pc);
> OK with these changes.
>
> Thanks.
>
thanks, all fixed, and committed.

-- 
Regards,
Jiong. Wang
Tilera Corporation.