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

2020-03-19 Thread Andrew Leonard
:[EXTERNAL] Re: Sponsor Request: 8241097: java/math/BigInteger/largeMemory/SymmetricRangeTests.java requires -XX:+CompactStrings The change has been pushed [1]. Brian [1] https://urldefense.proofpoint.com/v2/url?u=http-3A__hg.openjdk.java.net_jdk_jdk_rev_af221c1b1671&d=DwIFaQ&c=jf_iaS

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

2020-03-18 Thread Brian Burkhalter
ernet email: andrew_m_leon...@uk.ibm.com >>> >>> >>> >>> >>> From:Brian Burkhalter >>> To:Andrew Leonard >>> Cc:Java Core Libs >>> Date:17/03/2020 15:48 >>> Subject:[EXTE

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

2020-03-17 Thread Brent Christian
17/03/2020 15:48 Subject: [EXTERNAL] Re: Sponsor Request: 8241097: java/math/BigInteger/largeMemory/SymmetricRangeTests.java requires -XX:+CompactStrings To clarify, could the compact strings setting be detected and maybe add another @run tag and expect an OOME? Not sure this is cr

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

2020-03-17 Thread Brian Burkhalter
d Kingdom Ltd > internet email: andrew_m_leon...@uk.ibm.com > > > > > From:Brian Burkhalter > To: Andrew Leonard > Cc: Java Core Libs > Date: 17/03/2020 15:48 > Subject: [EXTERNAL] Re: Sponsor Request: 8241097: > java/

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

2020-03-17 Thread Andrew Leonard
ard Java Runtimes Development IBM Hursley IBM United Kingdom Ltd internet email: andrew_m_leon...@uk.ibm.com From: Brian Burkhalter To: Andrew Leonard Cc: Java Core Libs Date: 17/03/2020 15:48 Subject: [EXTERNAL] Re: Sponsor Request: 8241097: java/math/BigInteger/large

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

2020-03-17 Thread Brian Burkhalter
To clarify, could the compact strings setting be detected and maybe add another @run tag and expect an OOME? Not sure this is critical. Brian > On Mar 17, 2020, at 8:06 AM, Brian Burkhalter > wrote: > > You’re welcome. > > I see. Let’s leave this hang out a bit for further comment. Also, is

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

2020-03-17 Thread Brian Burkhalter
You’re welcome. I see. Let’s leave this hang out a bit for further comment. Also, is a test possible, or a modification of an existing test? Thanks, Brian > On Mar 17, 2020, at 7:52 AM, Andrew Leonard > wrote: > > Thanks Brian, > With Eclipse OpenJ9 you see this, which is very similar to i

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

2020-03-17 Thread Andrew Leonard
ew Leonard Cc: core-libs-dev@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 wh

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

2020-03-17 Thread Brian Burkhalter
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 > wrote: > > Please can I get a sponsor and reviews for the following "patch" to the > SymmetricRan

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

2020-03-17 Thread Andrew Leonard
Hi, 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