RFR: 8270179: Rename Amalloc_4 [v4]
Coleen Phillimore
coleenp at openjdk.java.net
Tue Jul 13 13:10:00 UTC 2021
On Mon, 12 Jul 2021 13:52:08 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> src/hotspot/share/memory/arena.hpp line 105:
>>
>>> 103: debug_only(void* malloc(size_t size);)
>>> 104:
>>> 105: void* internal_malloc_words(size_t x, AllocFailType alloc_failmode = AllocFailStrategy::EXIT_OOM) {
>>
>> If UseMallocOnly is retained, I think a better name for this would be internal_amalloc_only, to avoid confusion with the case of actually using malloc. (And yes, this can be taken as another argument for nuking that option.)
>
> Ok. I'll file another RFE for removing UseMallocOnly.
I decided that UseMallocOnly found an interesting bug once and there's a test for it, so it might find another interesting bug someday, so I added a comment and decided to leave it. We can decide some other time to remove it if we have a good alternative.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4750
More information about the hotspot-dev
mailing list