Quantcast

RFR 8179903: Clean up SPARC 32-bit support

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR 8179903: Clean up SPARC 32-bit support

George Triantafillou
Please review this fix to clean up SPARC 32-bit support.

JBS: https://bugs.openjdk.java.net/browse/JDK-8179903
webrev:
http://cr.openjdk.java.net/~gtriantafill/8179903-webrev/webrev/index.html 
<http://cr.openjdk.java.net/%7Egtriantafill/8179903-webrev/webrev/index.html>

This is a followup RFE to "JDK-8150388 Remove SPARC 32-bit support". The
work includes addressing formatting, 32-bit comments, and other issues
that Kim raised after JDK-8150388 was reviewed and checked in.

Built and tested on solaris-sparcv9-debug, solaris-x64-debug with the
nsk.jvmti, nsk.jdwp, and nsk.jdi testlists.

Thanks.

-George

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR 8179903: Clean up SPARC 32-bit support

George Triantafillou
Thanks Fred!

-George

On 5/12/2017 11:47 AM, Frederic Parain wrote:

> Looks good to me.
> Thank you for doing this clean up.
>
> Fred
>
>
>> On May 12, 2017, at 10:34, George Triantafillou <[hidden email]> wrote:
>>
>> Please review this fix to clean up SPARC 32-bit support.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8179903
>> webrev: http://cr.openjdk.java.net/~gtriantafill/8179903-webrev/webrev/index.html <http://cr.openjdk.java.net/%7Egtriantafill/8179903-webrev/webrev/index.html>
>>
>> This is a followup RFE to "JDK-8150388 Remove SPARC 32-bit support". The work includes addressing formatting, 32-bit comments, and other issues that Kim raised after JDK-8150388 was reviewed and checked in.
>>
>> Built and tested on solaris-sparcv9-debug, solaris-x64-debug with the nsk.jvmti, nsk.jdwp, and nsk.jdi testlists.
>>
>> Thanks.
>>
>> -George
>>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR 8179903: Clean up SPARC 32-bit support

George Triantafillou
In reply to this post by George Triantafillou
Thanks Jerry!

-George

On 5/12/2017 12:10 PM, Gerald Thornbrugh wrote:

> Hi George,
>
> Your changes look good to me.
>
> Jerry
>> On May 12, 2017, at 8:34 AM, George Triantafillou <[hidden email]> wrote:
>>
>> Please review this fix to clean up SPARC 32-bit support.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8179903
>> webrev: http://cr.openjdk.java.net/~gtriantafill/8179903-webrev/webrev/index.html <http://cr.openjdk.java.net/%7Egtriantafill/8179903-webrev/webrev/index.html>
>>
>> This is a followup RFE to "JDK-8150388 Remove SPARC 32-bit support". The work includes addressing formatting, 32-bit comments, and other issues that Kim raised after JDK-8150388 was reviewed and checked in.
>>
>> Built and tested on solaris-sparcv9-debug, solaris-x64-debug with the nsk.jvmti, nsk.jdwp, and nsk.jdi testlists.
>>
>> Thanks.
>>
>> -George
>>

Loading...