RFR: 8294308: Allow dynamically choosing the MEMFLAGS of a type without ResourceObj [v5]

[Coleen Phillimore]

On Mon, 3 Oct 2022 13:37:25 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Here's a suggested solution for the ticket mentioned and a use case for outputStream. I'm not attached to the name.
>> 
>> This saves space for all allocated outputStreams, which is nice. It also makes the purpose of ResourceObj more clear ("please handle the life cycle for me"), reducing the need for it.
>> 
>> Thank you for considering it.
>
> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add comment

src/hotspot/share/memory/allocation.hpp line 183:

> 181:   }
> 182: 
> 183:   static ALWAYSINLINE void* operator new(size_t size,

Were you going to move MEMFLAGS to the second argument in the new operators also?

src/hotspot/share/memory/allocation.hpp line 264:

> 262: 
> 263: // Dynamically pick the memory flags at allocation
> 264: class CHeapObjDynamic {

This can be simply using CHeapObjDynamic = CHeapObjImpl; so not to repeat all the various new operator declarations.

src/hotspot/share/utilities/ostream.hpp line 45:

> 43: // This allows for redirection via -XX:+DisplayVMOutputToStdout and
> 44: // -XX:+DisplayVMOutputToStderr
> 45: class outputStream : public CHeapObjDyn {

This should be CHeapObjDynamic.

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

PR: https://git.openjdk.org/jdk/pull/10412