[9] RFR (S): 8157181: Compilers accept modification of final fields outside initializer methods

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Fri Jun 10 15:09:41 UTC 2016


> Here is the updated webrev:
> http://cr.openjdk.java.net/~zmajo/8157181/webrev.09/

Looks good.

Last comment:

src/share/vm/interpreter/rewriter.cpp:

+           if (!(method->is_native() || method->is_abstract() || 
method->is_overpass())) {

Seems redundant. Native & abstract methods don't have body.
Shouldn't happen for overpasses as well (AFAIK they don't contain 
putstatic/putfield instructions), but it's better to analyze them (just 
in case).

Best regards,
Vladimir Ivanov


More information about the hotspot-dev mailing list