RFR: 8326962: C2 SuperWord: cache VPointer [v4]

[Emanuel Peter]

On Wed, 3 Apr 2024 21:50:10 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>>> One question: will VLoopAnalyzer default destructor clean up all memory used?
>> 
>> @vnkozlov there is no need, since it is all allocated over the `Arena` in `VLoopAnalyzer`:
>> 
>> 
>> // Arena for all submodules
>> Arena                _arena;
>> 
>> 
>> It is that arena that I pass into all submodules, such as `VLoopVPointer`. `VLoopAnalyzer` is stack allocated, so once the destructor removes its `_arena`, all submodules are also automatically deallocated.
>
>> It is that arena that I pass into all submodules, such as `VLoopVPointer`. `VLoopAnalyzer` is stack allocated, so once the destructor removes its `_arena`, all submodules are also automatically deallocated.
> 
> Good.

Thanks @vnkozlov @chhagedorn @jdksjolen for the reviews and suggestions!
@jdksjolen feel free to give me your ideas about Arena-allocation, I can still improve in a follow-up RFE ;)

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

PR Comment: https://git.openjdk.org/jdk/pull/18577#issuecomment-2036203197