On 5/26/11, Nathan Froyd <froy...@codesourcery.com> wrote:
> On 05/26/2011 10:24 PM, Lawrence Crowl wrote:
>> Index: gcc/cp/cp-objcp-common.c
>> ===================================================================
>> --- gcc/cp/cp-objcp-common.c (revision 174301)
>> +++ gcc/cp/cp-objcp-common.c (working copy)
>> @@ -99,6 +99,8 @@ cp_tree_size (enum tree_code code)
>>
>>      case TEMPLATE_INFO:         return sizeof (struct
>> tree_template_info);
>>
>> +    case TREE_BINFO:            return sizeof (struct tree_binfo);
>> +
>>      default:
>>        gcc_unreachable ();
>>      }
>
> This does not look right; TREE_BINFO is a variable-sized structure
> (and is not C++-specific in any event).

I'd forgotten that.

> Maybe you should be using tree_size instead of tree_code_size
> someplace?

Hm.  Possibly.  The PPH system is still mostly C, and so C++ parts
are being dropped, which could cause a different path further
upstream.

-- 
Lawrence Crowl

Reply via email to