Request for review (L): 7087838: JSR 292: add throttling logic for optimistic call site optimizations
Vladimir Kozlov
vladimir.kozlov at oracle.com
Fri Sep 16 10:50:29 PDT 2011
Looks fine to me.
Vladimir
Christian Thalinger wrote:
> [This change will be pushed after 7087357. So ignore the code removal in src/share/vm/classfile/javaClasses.cpp.]
>
> http://cr.openjdk.java.net/~twisti/7087838/
>
> 7087838: JSR 292: add throttling logic for optimistic call site optimizations
> Reviewed-by:
>
> The optimistic optimization for MutableCallSite and VolatileCallSite
> invalidate compiled methods on every setTarget. This possibly results
> in a recompile. For ever-changing call sites this is a performance
> hit.
>
> The fix is to add some throttling logic that prevents the optimistic
> optimization after a specified amount of invalidations per CallSite
> object.
>
> This change also moves the flush_dependents_on methods from Universe
> to CodeCache.
>
> src/share/vm/c1/c1_GraphBuilder.cpp
> src/share/vm/ci/ciCallSite.cpp
> src/share/vm/ci/ciCallSite.hpp
> src/share/vm/classfile/javaClasses.cpp
> src/share/vm/classfile/javaClasses.hpp
> src/share/vm/classfile/systemDictionary.cpp
> src/share/vm/classfile/vmSymbols.hpp
> src/share/vm/code/codeCache.cpp
> src/share/vm/code/codeCache.hpp
> src/share/vm/code/dependencies.cpp
> src/share/vm/code/dependencies.hpp
> src/share/vm/memory/universe.cpp
> src/share/vm/memory/universe.hpp
> src/share/vm/oops/methodOop.cpp
> src/share/vm/opto/callGenerator.cpp
> src/share/vm/opto/memnode.cpp
> src/share/vm/prims/jvmtiRedefineClasses.cpp
> src/share/vm/prims/methodHandles.cpp
> src/share/vm/runtime/globals.hpp
>
More information about the hotspot-compiler-dev
mailing list