On 6/26/18 5:18 PM, Poonam Parhar wrote:
Hello Coleen,

The changes look good to me. One minor thing that I noticed though:

At Line 94 in file src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/classfile/ClassLoaderData.java

The comment above the function find() is removed. Is that intentional?

Hi, Poonam, thank you for looking at this and your previous pre-review.  No, the comment was accidently removed and I'll restore it.  Thank you for catching this!

Coleen


Thanks,
Poonam

On 6/22/18 2:40 PM, coleen.phillim...@oracle.com wrote:
Summary: Modify SA code to not use SymbolTable and remove it.

This is to support the concurrent hashtable for SymbolTable.

open webrev at http://cr.openjdk.java.net/~coleenp/8205534.01/webrev
bug link https://bugs.openjdk.java.net/browse/JDK-8205534

Tested with hs-tier1-5.

Thanks,
Coleen


Reply via email to