RFR: 8362493: Cleanup CodeBuffer::copy_relocations_to

Manuel Hässig mhaessig at openjdk.org
Thu Jul 17 12:16:48 UTC 2025


On Thu, 17 Jul 2025 11:09:09 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this simple patch?
> 
> `CodeBuffer::copy_relocations_to(address buf, csize_t buf_limit, bool only_inst)` is only used in `copy_relocations_to(CodeBlob* dest)` which passes false to only_inst, so the former one should be able to be simplified.
> 
> Thank you!

Thank you for working on this cleanup, @Hamlin-Li! It looks good to me.

I kicked off some testing on our side and will let you know what the results are.

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

PR Review: https://git.openjdk.org/jdk/pull/26366#pullrequestreview-3029262121


More information about the hotspot-compiler-dev mailing list