Hi Serguei, Thanks for noticing that. It’s actually a duplicated bug ID for the same issue. I just fixed the web rev: http://cr.openjdk.java.net/~jiangli/8071962/webrev.00/.
Thanks, Jiangli On Jan 29, 2015, at 5:40 PM, serguei.spit...@oracle.com wrote: > On 1/29/15 5:13 PM, Jiangli Zhou wrote: >> Hi, >> >> Please review the change for JDK-8071962, "The SA code needs to be updated >> to support Symbol lookup from the shared archive”. >> >> In JDK-8059510, we introduced a compact form of hash table for the symbols >> in shared archive. The shared symbols are stored separately from the regular >> symbol table. The VM looks up both tables when searching for existing symbol >> at runtime. The SA code needs to do the same when looking up symbols. >> >> Webrev: http://cr.openjdk.java.net/~jiangli/8067977/webrev.00/ > > Jiangli, > > I'm not reviewing, just wanted to make sure there is no typo here ... > The webrev bug number is 8067977, but the RFR bug number is 8071962 which is > strange. > > Thanks, > Serguei > >> >> Tested on both 32-bit and 64-bit platforms: >> bin/java sun.jvm.hotspot.tools.DumpJFR <pid> >> >> JPRT tests in progress. >> >> Thanks, >> Jiangli >> >> >