RFR: 8024826: (s) : Remove alt-rt.jar, used by +AggressiveOps"

Mike Duigou mike.duigou at oracle.com
Tue Sep 17 16:34:06 UTC 2013


On Sep 17 2013, at 00:21 , David Holmes wrote:

> Hi Mike,
> 
> Looks like there is one test that knows it has to exercise the alt-rt.jar version:
> 
> ./java/util/TreeMap/Clone.java
> 
> * @run main/othervm Clone
> * @run main/othervm -XX:+AggressiveOpts Clone
> 
> but it's harmless to leave the second invocation.

I will remove it.

Since the +AggressiveOpts option remains it is still valid. A better general solution (that I hope someone is running) is to run the entire jtreg suite with +AggressiveOpts as an extra JDK option. 

> I didn't see anything in JDK or hotspot regression tests that would be impacted.
> 
> Once you are ready I will sponsor the hotspot patch for you through hotspot-rt.

Thank you.

Mike

> 
> Thanks,
> David
> 
> On 17/09/2013 12:37 PM, David Holmes wrote:
>> Reviewed both hotspot and jdk changes.
>> 
>> Thumbs up and good riddance! :)
>> 
>> How do you propose to handle the coordination of the push?
>> 
>> Thanks,
>> David
>> 
>> On 17/09/2013 10:16 AM, Mike Duigou wrote:
>>> Hello all;
>>> 
>>> This is a cross-repo patch which disables building and enabling of the
>>> alt-rt.jar file. The alt-rt.jar file has been used with the
>>> -XX:+AggressiveOpts option (which will be remaining for other
>>> optimizations). This jar file formerly contained (closed-source)
>>> versions of several JDK classes which were optimized for very specific
>>> workloads. The implementations weren't generally applicable and in
>>> many cases induced performance drops. In particular since we will not
>>> be providing optimized implementations of the Java 8 streams API for
>>> the alternate versions of these classes the performance relative to
>>> the standard implementations would be poor.
>>> 
>>> Over time we've (largely Alan Bateman) been trying to eliminate this
>>> difficult to support feature. With this changeset the alt-rt.jar
>>> feature is retired.
>>> 
>>> http://cr.openjdk.java.net/~mduigou/JDK-8024826/0/webrev/
>>> 
>>> Ideally I would like a reviewer for both the HotSpot and JDK repos
>>> (could be the same person).
>>> 
>>> Mike
>>> 




More information about the core-libs-dev mailing list