If the user doesn't initialize the field, its value should be 0 by default.
In that case, it's safe to place the data into .bss section.
2011/3/26 Sun Chan <sun.c...@gmail.com>
> but you are bypassing the FLD
> Sun
>
> On Sat, Mar 26, 2011 at 1:19 AM, Jian-Xin Lai <laij...@gmail.com> wrote:
> > 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
> >
>
--
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
