Integrated: 8256167: Convert JDK use of `Reference::get` to `Reference::refersTo`
On Thu, 3 Dec 2020 22:54:54 GMT, Mandy Chung wrote: > This patch replaces some uses of `Reference::get` to `Reference::refersTo` to > avoid keeping a referent strongly reachable that could cause unnecessary > delay in collecting such object. I only made change in some but not all > classes in core libraries when working with Kim on `Reference::refersTo`. > The remaining uses are left for the component owners to convert at > appropriate time. This pull request has now been integrated. Changeset: 972bc3b4 Author:Mandy Chung URL: https://git.openjdk.java.net/jdk/commit/972bc3b4 Stats: 60 lines in 12 files changed: 7 ins; 11 del; 42 mod 8256167: Convert JDK use of `Reference::get` to `Reference::refersTo` Reviewed-by: sspitsyn, shade, dfuchs, alanb, kbarrett - PR: https://git.openjdk.java.net/jdk/pull/1609
Re: RFR: 8257668: SA JMap - skip non-java thread stack dump for heap dump [v2]
On Sat, 5 Dec 2020 00:35:27 GMT, Lin Zang wrote: >> when use SA JMap dump commands, such as "jhsdb jmap --binaryheap" or "dump >> heap" with "jhsdb clhsdb", it keep printing "dumpStack: not java Thread.". >> Skip non-java thread stack dump to avoid printing the message. > > Lin Zang has updated the pull request incrementally with one additional > commit since the last revision: > > remove deadcode of println instead of change in HeapHprofBinWriter Marked as reviewed by phh (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/1593
RFR: 8256424: Move ciSymbol::symbol_name() to ciSymbols::symbol_name()
This moves the mirroring of vmSymbols in ciSymbols to a separate file, ciSymbols.hpp, to reduce includes throughout hotspot (and clean up the ciSymbol namespace). In a few places code is moved from .hpp to .cpp to facilitate this. With PCH disabled, this reduces total includes from 276949 to 276332 - Commit messages: - remote merge - Prepare for 1237 changes - Adjust includes after vmIntrinsic changes - Merge master - Outline is_call_site_target to remove include from ciField.hpp - Outline is_autobox_cache to remove include from ciField.hpp - Merge branch 'master' into ciSymbols - Fix bad definition - Define debug-only sid_ok in .cpp to avoid need to include vmSymbols.hpp in ciSymbol.hpp - Revert accidental include order swap - ... and 1 more: https://git.openjdk.java.net/jdk/compare/86b65756...b05957b8 Changes: https://git.openjdk.java.net/jdk/pull/1256/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=1256=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8256424 Stats: 174 lines in 32 files changed: 102 ins; 30 del; 42 mod Patch: https://git.openjdk.java.net/jdk/pull/1256.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1256/head:pull/1256 PR: https://git.openjdk.java.net/jdk/pull/1256