On Thu, 7 Oct 2021 06:12:07 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:
>> Leo Korinth has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Use a thread local buffer so that the compiler might reorder operator new.
>
> src/hotspot/share/memory/allocation.hpp line 398:
>
>> 396: class ResourceObj ALLOCATION_SUPER_CLASS_SPEC {
>> 397: public:
>> 398: enum allocation_type : uint8_t { STACK_OR_EMBEDDED, RESOURCE_AREA,
>> C_HEAP, ARENA };
>
> Consider adding an "empty" allocation-type, that indicates an entry in the
> RecentAllocations is unused.
After thinking a bit, I will not add an "empty" value. It will make it harder
to make functions "total" that matches on the enum, and the "empty" value does
not really need to be reset in RecentAllocations.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5387
