On Tue, 9 Dec 2025 12:29:28 GMT, Stefan Karlsson <[email protected]> wrote:

> There are a few left-over blank line changes, redundant includes, and unused 
> code in the memory/ directory. I suggest that we clean those out to make it 
> nicer to review the patch.

It is a remain of an experiment to use a small thread-local area to store 
buffered values, which are often short lived, with a kind of automatic 
reclamation based on frame exits (I'm simplifying here). The idea was to avoid 
filling the Java heap with those buffered values. It worked but performance was 
bad. Browsing interpreter frames to find which values in this thread-local area 
were still alive was expensive, and the ratio cost/amount_of_memory_freed was 
not good. At the end, the Java heap GC was more efficient to handle those short 
lived buffered values, especially with generational GCs.

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

PR Comment: https://git.openjdk.org/valhalla/pull/1781#issuecomment-3632909644

Reply via email to