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