Comp bug due to windows compiler upgrades (original) (raw)

Alejandro E Murillo alejandro.murillo at oracle.com
Wed Apr 22 15:48:54 UTC 2015

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

More information about the hotspot-dev mailing list