RFR(M): 8139040: Fix initializations before ShouldNotReachHere()
Coleen Phillimore
coleen.phillimore at oracle.com
Mon Oct 12 18:03:03 UTC 2015
Hi, There are surprisingly few changes to runtime code for this, which
look fine. I'm adding the compiler alias though.
Thanks,
Coleen
On 10/7/15 9:22 AM, Lindenmaier, Goetz wrote:
> Hi,
>
> SAP requires us to fix a row of issues in the hotspot coding. I would like
> to share these with openJDK.
>
> This webrev fixes a row of missing intializations, mostly combined with ShouldNotReachHere()
> in default cases of switches or the like.
> http://cr.openjdk.java.net/~goetz/webrevs/8139040-init/webrev.00/
>
> In the debug build, ShouldNotReachHere() can be suppressed, so the uninitialized value actually can cause problems.
> In opt builds, not all tools recognize the ShouldNotReachHere properly.
>
> In addition to this I would like to add -Wuninitialized to the warning flags.
> This finds most of these issues in the opt build and
> would require an additional 70 fixes plus fixes in jvmtiEnter.xsl.
> Would it be appreciated to set this flag?
>
> Best regards,
> Goetz.
>
>
>
More information about the hotspot-compiler-dev
mailing list