RFR (M): 8000740: remove LinkWellKnownClasses
John Rose
john.r.rose at oracle.com
Thu Oct 11 12:00:01 PDT 2012
The change in methodHandles.cpp is not under -XX:+LinkWellKnownClasses.
As the comment says, it is intended to be an optional optimization for 292 users.
The short-cut for Object (and maybe Class) is probably necessary for bootstrapping.
I don't think there will be a regression there, but let's be watchful.
Code removal is good... Reviewed.
— John
On Oct 10, 2012, at 5:10 PM, Christian Thalinger wrote:
> http://cr.openjdk.java.net/~twisti/8000740/
>
> 8000740: remove LinkWellKnownClasses
> Reviewed-by:
>
> With 8000263 LinkWellKnownClasses was turned on by default to see if
> it works. Nightly testing showed that various JDI and JVMTI tests fail
> with LinkWellKnownClasses turned on, e.g.:
>
> jdk/test/com/sun/jdi/NewInstanceTest.java
>
> To avoid future problems we remove the code altogether.
>
> Additionally I resurrected some 292 code that is required when
> building with JDK 6.
>
> src/share/vm/classfile/classFileParser.cpp
> src/share/vm/classfile/classFileParser.hpp
> src/share/vm/classfile/systemDictionary.cpp
> src/share/vm/classfile/systemDictionary.hpp
> src/share/vm/prims/methodHandles.cpp
> src/share/vm/runtime/globals.hpp
>
More information about the hotspot-compiler-dev
mailing list