Looks all right. Thanks,
Brian > On Mar 26, 2019, at 9:26 AM, Thomas Stüfe <thomas.stu...@gmail.com> wrote: > > Looks fine and trivial. > > ..Thomas > > On Tue, Mar 26, 2019 at 2:02 PM Aleksey Shipilev <sh...@redhat.com> wrote: > >> Ping, no takers? >> >> -Aleksey >> >> On 3/25/19 3:33 PM, Aleksey Shipilev wrote: >>> Bug: >>> https://bugs.openjdk.java.net/browse/JDK-8221401 >>> >>> My major concern is to make tier1 passing on x86_32. This is one of the >> tests that fail. Smaller >>> heap size does not fit this test case, unfortunately. So, we have to >> disable it for 32-bit builds. >>> In fact, somebody did it already for 8u backport, but not upstream! >>> >>> Fix: >>> >>> diff -r 59816a32df2c >> test/jdk/java/math/BigInteger/LargeValueExceptions.java >>> --- a/test/jdk/java/math/BigInteger/LargeValueExceptions.java Mon Mar >> 25 13:12:37 2019 +0100 >>> +++ b/test/jdk/java/math/BigInteger/LargeValueExceptions.java Mon Mar >> 25 15:29:16 2019 +0100 >>> @@ -23,11 +23,11 @@ >>> >>> /* >>> * @test >>> * @bug 8200698 >>> * @summary Tests that exceptions are thrown for ops which would >> overflow >>> - * @requires os.maxMemory >= 4g >>> + * @requires (sun.arch.data.model == "64" & os.maxMemory >= 4g) >>> * @run testng/othervm -Xmx4g LargeValueExceptions >>> */ >>> import java.math.BigInteger; >>> import static java.math.BigInteger.ONE; >>> import org.testng.annotations.Test; >>> >>> >>> Testing: Linux {x86_64, x86_32} fastdebug build and test, jdk-submit >> (running) >>> >>> Thanks, >>> -Aleksey >>> >> >> >> -- >> Thanks, >> -Aleksey >> >> ------------------- >> Red Hat GmbH, http://www.de.redhat.com/, Registered seat: Grasbrunn, >> Commercial register: Amtsgericht Muenchen, HRB 153243, >> Managing Directors: Charles Cachera, Michael O'Neill, Tom Savage, Eric >> Shander