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

Reply via email to