Request for reviews (S): 7085404: JSR 292: VolatileCallSites should have push notification too

Vladimir Kozlov vladimir.kozlov at oracle.com
Thu Sep 1 08:37:51 PDT 2011


Looks good.

Vladimir

On 9/1/11 2:17 AM, Christian Thalinger wrote:
> Here is the official review for 7085404.  Tom has already reviewed this change in another thread on this list.
>
> http://cr.openjdk.java.net/~twisti/7085404/
>
> 7085404: JSR 292: VolatileCallSites should have push notification too
> Reviewed-by: never
>
> 7071653 implemented push notification for ConstantCallSites and
> MutableCallSites.  This should be extended to VolatileCallSites too.
>
> src/share/vm/c1/c1_GraphBuilder.cpp
> src/share/vm/ci/ciField.hpp
> src/share/vm/interpreter/interpreterRuntime.cpp
> src/share/vm/opto/callGenerator.cpp
> src/share/vm/opto/doCall.cpp
> src/share/vm/opto/parse3.cpp
> src/share/vm/prims/unsafe.cpp
>


More information about the hotspot-compiler-dev mailing list