Thanks a lot for the review Mandy, Igor, Harsha .
-Ujwal.
On 6/14/2017 10:38 PM, Igor Ignatyev wrote:
looks good to me as well.
-- Igor
On Jun 14, 2017, at 10:06 AM, Ujwal Vangapally
<[email protected] <mailto:[email protected]>> wrote:
Hi Igor can you kindly review this.
Thanks,
Ujwal
On 6/14/2017 9:40 AM, Mandy Chung wrote:
On Jun 13, 2017, at 8:13 PM, Ujwal Vangapally
<[email protected] <mailto:[email protected]>>
wrote:
Thanks for the review Mandy.
updated webrev with simplified version using @requires
os.simpleArch=="x64".
can't use @requires (os.family != "windows") | (os.simpleArch !=
"i586")alone as sometimes it might fail on Linux 32bit as well
webrev :
http://cr.openjdk.java.net/~uvangapally/webrev/2017/8178508/webrev.03
This version looks good.
Mandy
