David, thx for the help. yiran, please go ahead
sun On Wed, Apr 6, 2011 at 8:28 AM, David Coakley <dcoak...@gmail.com> wrote: > Hi Yiran, > > This change looks right to me, but I am not a gatekeeper for this part > of the compiler. > > If you have a test case showing the consequences of the bug, I am sure > that would help get approval. > > -David Coakley / AMD Open Source Compiler Engineering > > On Wed, Mar 30, 2011 at 6:48 AM, Yiran Wang <yiran.w...@gmail.com> wrote: >> Hi, >> >> Could a gatekeeper please review this patch? >> >> When the eh-range get flattened, this specific eh-range generated is >> part of the original range "j", rather than "i". >> >> Best Regards, >> yiran >> >> Index: osprey/be/cg/eh_region.cxx >> =================================================================== >> --- osprey/be/cg/eh_region.cxx (revision 1192) >> +++ osprey/be/cg/eh_region.cxx (revision 1193) >> @@ -1122,7 +1122,7 @@ >> last_child = first_child = i; >> if ( first_child != no_child && i != last_child ) >> { >> - EH_RANGE new_range (range_list[i].rid); >> + EH_RANGE new_range (range_list[j].rid); >> new_range.start_label = range_list[last_child].end_label; >> new_range.end_label = range_list[i].start_label; >> new_range.end_bb = Get_Label_BB(range_list[i].start_label); >> >> ------------------------------------------------------------------------------ >> 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 >> > > ------------------------------------------------------------------------------ > Xperia(TM) PLAY > It's a major breakthrough. An authentic gaming > smartphone on the nation's most reliable network. > And it wants your games. > http://p.sf.net/sfu/verizon-sfdev > _______________________________________________ > Open64-devel mailing list > Open64-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/open64-devel > ------------------------------------------------------------------------------ Xperia(TM) PLAY It's a major breakthrough. An authentic gaming smartphone on the nation's most reliable network. And it wants your games. http://p.sf.net/sfu/verizon-sfdev _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel