On Fri, 12 Dec 2025 15:28:27 GMT, Stefan Johansson <[email protected]> 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 <[email protected]> Committer: Paul Hübner <[email protected]> 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
