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 >
------------------------------------------------------------------------------ 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
