RFR: 8269736: Optimize CDS PatchEmbeddedPointers::do_bit()

Calvin Cheung ccheung at openjdk.org
Fri Dec 16 22:10:58 UTC 2022


On Tue, 25 Oct 2022 20:56:10 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

> The current implementation reads the narrow oop, decodes it using the "dump time" encoding, encodes it using "runtime" encoding, and then stores it back into the heap. In the cases where the dumptime and runtime shifts are equal, there is an opportunity to calculate the fixed distance between oops so they can be patched more optimally. Verified with tier 1, 2, and 4 tests.

Changes seem ok but you'll need to fix the minimal and arm build failures.
I'm guessing the include of `utilities/bitMap.hpp` is missing in `archiveHeapLoader.hpp.`

Also, which tiers testing have you run with this change?

-------------

PR: https://git.openjdk.org/jdk/pull/10863


More information about the hotspot-runtime-dev mailing list