RFR: 8335701: Make GrowableArray templated by an Index [v2]

Johan Sjölen jsjolen at openjdk.org
Fri Jul 26 12:46:48 UTC 2024


On Thu, 4 Jul 2024 14:07:57 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> If this is for src/hotspot/share/nmt/arrayWithFreeList.hpp, would it not be a lot simpler to just implement it there, and give it another backing store?
> 
> In particular because after doing all this work it still won't even support the feature I was hoping for, mainly the ability to put an indexed free list atop of existing memory.

I did that first and it sure is simpler, but I'm not sure whether it's a good idea to have to support such a backing storage. See `resizable_array` in here: https://github.com/openjdk/jdk/pull/20002

 Still, it does do what you asked for, kind of, see: `GrowableArrayFromArray`. I can adapt AWFL to be able to use either `GAFA` or `GA`.

It's also not the only reason to do this.

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

PR Comment: https://git.openjdk.org/jdk/pull/20031#issuecomment-2209118236


More information about the serviceability-dev mailing list