This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [pph] More C++ Tree Nodes (issue4526083)


On 5/27/11, dnovillo@google.com <dnovillo@google.com> wrote:
>
> http://codereview.appspot.com/4526083/diff/1/gcc/cp/cp-objcp-common.c
> File gcc/cp/cp-objcp-common.c (right):
>
> http://codereview.appspot.com/4526083/diff/1/gcc/cp/cp-objcp-common.c#newcode103
> gcc/cp/cp-objcp-common.c:103:
>       case TEMPLATE_INFO:         return sizeof (struct
> tree_template_info);
>
> +    case TREE_BINFO:            return sizeof (struct tree_binfo);
> +
>
>
> TREE_BINFO is not a C++ code and it's variably-sized.  It is already
> handled by tree.c:tree_size()
>
> Not sure why you needed to add this here?

It appears on of the other fixes in that patch fixed the issue,
so I will remove this change.

>
> http://codereview.appspot.com/4526083/
>


-- 
Lawrence Crowl


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]