<div dir="auto">Johan, I would just remove the noncopyable. Its questionable anyway, there is no reason one shouldnt be able to copy the AWFL, in particular because client code just uses indices, not pointers.</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue 25. Jun 2024 at 21:01, Sonia Zaldana Calles <<a href="mailto:szaldana@openjdk.org">szaldana@openjdk.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On Tue, 25 Jun 2024 17:57:36 GMT, Johan Sjölen <<a href="mailto:jsjolen@openjdk.org" target="_blank">jsjolen@openjdk.org</a>> wrote:<br>
<br>
>> Thank you for the in-depth reviewing of this change.<br>
><br>
>> Hi @jdksjolen, I'm seeing some build failures after this PR got integrated. I filed [JDK-8335108](<a href="https://bugs.openjdk.org/browse/JDK-8335108" rel="noreferrer" target="_blank">https://bugs.openjdk.org/browse/JDK-8335108</a>) for reference.<br>
> <br>
> 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: <a href="https://godbolt.org/z/eK5GaTceY" rel="noreferrer" target="_blank">https://godbolt.org/z/eK5GaTceY</a><br>
> <br>
> 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 :-).<br>
> <br>
> There's something weird with the fact that you get builds with C++20 support enabled, not sure what your configure options are.<br>
<br>
Hi @jdksjolen, thanks for looking into it! <br>
<br>
> 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: <a href="https://godbolt.org/z/eK5GaTceY" rel="noreferrer" target="_blank">https://godbolt.org/z/eK5GaTceY</a><br>
> <br>
> 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 :-).<br>
<br>
Makes sense. I'll submit a follow-up PR to fix this. <br>
<br>
> There's something weird with the fact that you get builds with C++20 support enabled, not sure what your configure options are.<br>
<br>
I'm not sure. I don't have any special configure options. I came across this error after a plain ```bash configure``` but I guess my installed g++ version is 14.1.1.<br>
<br>
-------------<br>
<br>
PR Comment: <a href="https://git.openjdk.org/jdk/pull/18979#issuecomment-2189747196" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/18979#issuecomment-2189747196</a><br>
</blockquote></div></div>