ok.
David
On Tue, Sep 29, 2015 at 4:08 PM, Rong Xu wrote:
> You are right. I attached the updated patch to this email.
>
> On Tue, Sep 29, 2015 at 3:10 PM, Xinliang David Li wrote:
>>else
>> {
>>gfi_ptr = gi_ptr->functions[f_ix];
>> - if (gfi_ptr && gfi_ptr
You are right. I attached the updated patch to this email.
On Tue, Sep 29, 2015 at 3:10 PM, Xinliang David Li wrote:
>else
> {
>gfi_ptr = gi_ptr->functions[f_ix];
> - if (gfi_ptr && gfi_ptr->key == gi_ptr)
> + if (gfi_ptr)
> length = GCO
else
{
gfi_ptr = gi_ptr->functions[f_ix];
- if (gfi_ptr && gfi_ptr->key == gi_ptr)
+ if (gfi_ptr)
length = GCOV_TAG_FUNCTION_LENGTH;
- else
-length = 0;
}
The removal of 'else' path seems wrong.
David
On Tue
