[8u] RFR: 8150688: Fix os_windows siglabel

Kim Barrett kim.barrett at oracle.com
Mon Jun 18 23:42:03 UTC 2018


> On Jun 15, 2018, at 11:41 AM, Kevin Walls <kevin.walls at oracle.com> wrote:
> 
> Hi,
> 
> I'd like to get a review of this backport from 9 to 8u:
> 
> 8150688: Fix os_windows siglabel
> JBS:https://bugs.openjdk.java.net/browse/JDK-8150688
> 
> 9 changeset:
> URL:http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/80706cc25494
> 
> 9 review thread:
> http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-March/018236.html
> 
> Proposed 8u change:http://cr.openjdk.java.net/~kevinw/8150688/webrev.00/

2228     , def_excpt(EXCEPTION_REG_NAT_CONSUMPTION),

stray trailing comma.

Otherwise, looks good.  But note that I am not a JDK 8 reviewer (or even committer).

> This is a pretty clean backport. but didn't import automatically: in 8u
> there is no "os::get_signal_number()" and some of the def_excpt... lines
> didn't apply automatically, but they are the same change.
> 
> This removes some errors when varying the windows compiler in use, still
> builds and tests OK on current standard VS compiler.
> 
> Thanks!
> Kevin




More information about the hotspot-runtime-dev mailing list