Joel, Paul,

thanks for reviewing!

Pushed.

/Claes

On 01/15/2015 02:46 PM, Joel Borggrén-Franck wrote:
Looks good.

cheers
/Joel

On 15 jan 2015, at 13:32, Claes Redestad <claes.redes...@oracle.com> wrote:

Hi,

please review this patch to use double-checked locking to avoid
synchronization overhead on repeated access to the
declaredAnnotations field in Executable and Field.

webrev: http://cr.openjdk.java.net/~redestad/8068736/webrev.00
bug: https://bugs.openjdk.java.net/browse/JDK-8068736

Testing: jtreg java/lang

Benchmarking:
- no significant changes in a small set of SPEC benchmarks
- up to 4x in micros focused on repeated access to the
declaredAnnotations field, both in single- and multithreaded setups
- less than 5% cost (not statistically significant) on micros accessing
the declaredAnnotation map of a method only once.

/Claes

Reply via email to