[lworld] Integrated: 8373608: [lworld] Remove ObjBufferAllocator

Stefan Johansson sjohanss at openjdk.org
Wed Dec 17 11:44:45 UTC 2025


On Fri, 12 Dec 2025 15:28:27 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

> Please review this change to remove `ObjBufferAllocator` since the impementation now is identical to `ObjAllocator`. 
> 
> **Summary**
> This was not the case when ObjBufferAllocator was first introduced. Back then it skipped clearing the memory and just called oopDesc::set_klass_gap(mem, 0). This is no longer the case after [JDK-8350630](https://bugs.openjdk.org/browse/JDK-8350630) in which `ObjBufferAllocator` was changed to also clear the memory. The previous optimization is likely not needed and we should streamline the code to always use `ObjAllocator`.
> 
> **Testing**
> * mach5 tier1-3 (ongoing)

This pull request has now been integrated.

Changeset: 619ef43b
Author:    Stefan Johansson <sjohanss at openjdk.org>
Committer: Paul Hübner <phubner at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/619ef43b00b1e13452a2dd8f4d8320f45e27f8e0
Stats:     34 lines in 7 files changed: 0 ins; 32 del; 2 mod

8373608: [lworld] Remove ObjBufferAllocator

Reviewed-by: stefank, phubner

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

PR: https://git.openjdk.org/valhalla/pull/1794


More information about the valhalla-dev mailing list