Comp bug due to windows compiler upgrades

[Alejandro E Murillo]

Thanks Vladimir

On 4/22/2015 9:05 AM, Vladimir Kozlov wrote:
> 8077590/webrev/ fix looks good.
> I did not see webrev for 8015396.
this one is still active, Christian provided a patch for the test
to be excluded until that one is fixed. Look for this:
RFR(XS): 8077994 - [TESTBUG] Exclude compiler/floatingpoint/ModNaN.java

Thanks!
Alejandro
>
> 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

-- 
Alejandro