Yes. The function Is_Aggregate_Init_Zero_Struct() will call
Is_Aggregate_Init_Zero() if meet a nested data structure (nested struct or
array).

2011/3/26 Sun Chan <sun.c...@gmail.com>

> will your fix handle nested structs correctly?
> Sun
>
> On Fri, Mar 25, 2011 at 9:33 PM, Jian-Xin Lai <laij...@gmail.com> wrote:
> > Hi,
> >
> > Could a gate keeper review the patch for #746?
> > For the first case, the TY for c4 is:
> > [54]: c4            : (f: 0x1800 return_in_mem content_seen) size 4 M:
> > STRUCT
> >              0  arg             .predef_I4 (#4) align 4
> >                 fl:0x0001 last_field
> > For the thrid case, the TY for c4 is:
> > [57]: c4            : (f: 0x1800 return_in_mem content_seen) size 4 M:
> > STRUCT
> >              0  .anonymous.4    c0 (#55) align 4
> >                 fl:0x0180
> >              0  .anonymous.5    c1 (#58) align 4
> >                 fl:0x0180
> >              0  m0              .predef_I4 (#4) align 4
> >                 fl:0x0001 last_field
> >
> > In GCC tree, there are a lot of empty base classes. When we trying to
> > traverse the GCC tree and advance the FLD in the TY, we don't ignore
> these
> > empty class correctly. The segmentation fault happens wwhen we go beyond
> the
> > last field of the TY. My patch is to remove all WHIRL FLD and only
> traverse
> > the GCC tree to find out if all fields are initialized by zero.
> >
> > Hi Suneel,
> > Is this patch going to be included in open64 4.2.4 release or not? Thank
> you
> > very much.
> >
> > --
> > Regards,
> > Lai Jian-Xin
> >
> >
> ------------------------------------------------------------------------------
> > Enable your software for Intel(R) Active Management Technology to meet
> the
> > growing manageability and security demands of your customers. Businesses
> > are taking advantage of Intel(R) vPro (TM) technology - will your
> software
> > be a part of the solution? Download the Intel(R) Manageability Checker
> > today! http://p.sf.net/sfu/intel-dev2devmar
> > _______________________________________________
> > Open64-devel mailing list
> > Open64-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/open64-devel
> >
> >
>



-- 
Regards,
Lai Jian-Xin
------------------------------------------------------------------------------
Enable your software for Intel(R) Active Management Technology to meet the
growing manageability and security demands of your customers. Businesses
are taking advantage of Intel(R) vPro (TM) technology - will your software 
be a part of the solution? Download the Intel(R) Manageability Checker 
today! http://p.sf.net/sfu/intel-dev2devmar
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to