RFR: 8333658: NMT: Use an allocator with 4-byte pointers to save memory in NativeCallStackStorage [v34]

[Johan Sjölen]

On Tue, 25 Jun 2024 14:36:28 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Do not use char array
>
> Thank you for the in-depth reviewing of this change.

> Hi @jdksjolen, I'm seeing some build failures after this PR got integrated. I filed [JDK-8335108](https://bugs.openjdk.org/browse/JDK-8335108) for reference.

Hi Sonia, thanks for contacting me. It seems like this is a C++20 issue, but Hotspot only supports up to C++14. Here's a Godbolt for reproducing the issue, change the `-std=c++14` to `-std=c++20` to see the issue: https://godbolt.org/z/eK5GaTceY

That ought to mean that I don't have to back out this PR and re-introduce it with the bug fixed, but I still think that we should fix the bug because we do want to get to C++20 some day :-).

There's something weird with the fact that you get builds with C++20 support enabled, not sure what your configure options are.

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

PR Comment: https://git.openjdk.org/jdk/pull/18979#issuecomment-2189625153