RFR (S) 8223657: Remove unused THREAD argument from SymbolTable functions

gerard ziemski gerard.ziemski at oracle.com
Fri May 10 16:22:31 UTC 2019


hi Coleen,

Looks OK (though I wouldn't call it Small, Simple yes ;-)

One question: in src/hotspot/share/jvmci/compilerRuntime.cpp, did you 
mean to remove the line:

   48       TempNewSymbol sym = SymbolTable::new_symbol(name, len, CHECK);

?

Good cleanup.


cheers



On 5/10/19 10:15 AM, coleen.phillimore at oracle.com wrote:
> Summary: also made lookup and lookup_only functions private to 
> SymbolTable.  External callers use new_symbol or probe.
>
> Ran hs-tier1-3.
>
> open webrev at http://cr.openjdk.java.net/~coleenp/2019/8223657.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8223657
>
> Thanks,
> Coleen



More information about the hotspot-runtime-dev mailing list