looks good to me as well.
-- Igor
> On Jun 14, 2017, at 10:06 AM, Ujwal Vangapally <[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
>>> <http://cr.openjdk.java.net/%7Euvangapally/webrev/2017/8178508/webrev.03>
>>>
>>
>> This version looks good.
>>
>> Mandy
>>
>
