Comp bug due to windows compiler upgrades

[Vladimir Kozlov]

8077590/webrev/ fix looks good.
I did not see webrev for 8015396.

Thanks,
Vladimir

On 4/22/15 7:44 AM, Alejandro E Murillo wrote:
> Hi Nils, and others, one more thing,
> there's not a reviewer in that patch.
> to gain sometime here, can any reviewer in  this thread review the change
> and agree to be added as a reviewer?
>
> Thanks
> Alejandro
>
> On 4/22/2015 5:37 AM, Alejandro E Murillo wrote:
>>
>>
>> On 4/22/2015 5:26 AM, Nils Eliasson wrote:
>>> I started with removing optimization from all sharedRuntime* files as you suggested and it worked. I got a new
>>> warning (as error) in sharedRuntime.cpp " overflow in constant arithmetic"
>>> https://msdn.microsoft.com/en-us/library/f4s2sw05.aspx. That warning pointed to the high(double) and low(double)
>>> functions. So I tested if it was them causing the failure and it worked too.
>>>
>>> Here is a webrev: http://cr.openjdk.java.net/~neliasso/8077590/webrev/
>>>
>>> Alejandro - is it still jdk9/dev I should push to?
>> yes, but this need to be pushed via jprt along with Christian's change for JDK-8015396,
>> otherwise the jprt job will fail.
>> Christian, if you provide me with your committed patch and I can push both patches
>> or if you prefer, you can grab Nils patch from the webrev and submit a job with your
>> changes  along with that to jdk9/dev? just let me know
>>
>> Thanks!
>> Alejandro