Yes, looks good. Thanks, -Aleksey
On Thu, Aug 11, 2016 at 2:20 AM, Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote: > Okay, I simplified that confusing part of the fix and made a minimal > change to minimize chances of any unintended changes in behavior. > > http://cr.openjdk.java.net/~igerasim/8163518/02/webrev/ > > Is it good to push now? > > Btw, that @requires (os.simpleArch == "x64" & os.maxMemory > XX) logic > works quite well. > I've seen that on different hosts one, two or all three tests were run, > depending on the environment. > I think, we may consider enabling some of the memory-demanding tests that > are currently ignored. > > With kind regards, > Ivan > >
