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

Reply via email to