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

Reply via email to