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

Christian Thalinger christian.thalinger at oracle.com
Thu Sep 1 09:02:31 PDT 2011


Thank you, Vladimir.  -- Christian

On Sep 1, 2011, at 5:37 PM, Vladimir Kozlov wrote:

> 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