[9] RFR(S): 8043913: remove legacy code in SPARC's VM_Version::platform_features

[Zoltan Majo]

Hi,


On 12.08.2014 18:23, Christian Thalinger wrote:
> On Aug 12, 2014, at 12:12 AM, Zoltán Majó <zoltan.majo at oracle.com> wrote:
>
>> Hi,
>>
>>
>> please remove the following patch.
>>
>> Problem: Since on SPARC we are now building/running on Solaris 10 and up, legacy code in the method VM_Version::platform_features can be removed.
>>
>> Solution: The getisax(2) call is available from Solaris 10, therefore legacy code to determine platform features can be removed. Solaris 10 defines some of the AV* constants used in the method. Some of these defines could have been removed as well, but different versions of Solaris 10 define different subsets of the constants, therefore it is better to leave the defines for now.
> Can you file an Enhancement to keep track of this?

Issue 8043913 is currently filed as an enhancement. Did you mean to file 
an enhancement for the time when we decide to drop support for Solaris < 
11 (and add some notes there on what could still be done)? Or should I 
add notes already to the current issue?

Thank you!

Best,


Zoltan


>> Webrev: http://cr.openjdk.java.net/~thartmann/8043913/webrev.00/
>>
>> Testing:
>>
>> - built JVM on solaris_sparc_5.10u5 and on solaris_sparc_5.11.1;
>> - ran all JPRT tests on the solaris_sparcv9* platform;
>> - ran test/sanity/ExecuteInternalVMTests.java (the test executed by the "hotspot_compiler" JPRT test) on solaris_sparc_5.11.1, solaris_sparc_5.10u5, and solaris_sparc_5.10u6
>>
>> Thank you and best regards,
>>
>>
>> Zoltan
>>