Sponsor Request: 8241097: java/math/BigInteger/largeMemory/SymmetricRangeTests.java requires -XX:+CompactStrings

[Andrew Leonard]

Thanks Brian,
With Eclipse OpenJ9 you see this, which is very similar to if you force 
Hotspot with -XX:-CompactStrings:
Exception in thread "main" java.lang.OutOfMemoryError
        at 
java.base/java.lang.AbstractStringBuilder.hugeCapacity(AbstractStringBuilder.java:214)
        at 
java.base/java.lang.AbstractStringBuilder.newCapacity(AbstractStringBuilder.java:206)
        at 
java.base/java.lang.AbstractStringBuilder.ensureCapacityInternal(AbstractStringBuilder.java:173)
        at 
java.base/java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:748)
        at 
java.base/java.lang.StringBuilder.append(StringBuilder.java:241)
        at StringBufferTest.main(StringBufferTest.java:8)

Cheers
Andrew

Andrew Leonard
Java Runtimes Development
IBM Hursley
IBM United Kingdom Ltd
internet email: andrew_m_leonard at uk.ibm.com 




From:   Brian Burkhalter <brian.burkhalter at oracle.com>
To:     Andrew Leonard <andrew_m_leonard at uk.ibm.com>
Cc:     core-libs-dev at openjdk.java.net
Date:   17/03/2020 14:48
Subject:        [EXTERNAL] Re: Sponsor Request: 8241097: 
java/math/BigInteger/largeMemory/SymmetricRangeTests.java requires 
-XX:+CompactStrings



Hi Andrew,

I can help you. The change looks good but what do you see when you run the 
test without the change on a non-Hotspot VM?

Thanks,

Brian

On Mar 17, 2020, at 7:36 AM, Andrew Leonard <andrew_m_leonard at uk.ibm.com> 
wrote:

Please can I get a sponsor and reviews for the following "patch" to the 
SymmetricRangeTests.java test, for bug 
https://bugs.openjdk.java.net/browse/JDK-8241097
patch: http://cr.openjdk.java.net/~aleonard/8241097/webrev.00/

The current testcase makes the assumption that -XX:+CompactStrings is the 
default, which it is for Hotspot, but that's not necessarily true for all 
JVM providers.



Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU