<div dir="ltr"><span style="color:rgb(80,0,80)">Yes, that looks like a cleaner approach. JDK-8282322 implementation just focused on keeping the patch simple. There are no concerns about really old GCCs. it simply would be nice to have a way to apply the change(s) to 17 which is buildable with recent enough versions.</span><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 8, 2022 at 7:35 AM Kim Barrett <<a href="mailto:kim.barrett@oracle.com">kim.barrett@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sep 7, 2022, at 5:30 AM, Dmitry Chuyko <<a href="mailto:dmitry.chuyko@bell-sw.com" target="_blank">dmitry.chuyko@bell-sw.com</a>> wrote:<br>
> <br>
> Hi Kim,<br>
> <br>
> The implementation of the main JDK-8261027 change doesn't just try to provide the functionality when built with older GCC (or not GCC). What it does is dynamically switch to a more advanced implementation if the appropriate hardware capabilities were detected during the VM start. The selected code is picked by the compiler. So there are 2 implementations and non-LSE one is the default.  This allows us to provide a single binary for all supported ARM devices and get better performance where possible.<br>
<br>
That's close to what the __atomic intrinsics with -moutline-atomics do. Except<br>
the compiler intrinsic approach has various benefits accrued from being built<br>
into the compiler.  (And the code that we need to maintain is much simpler.)<br>
<br>
> From another hand, a less advanced implementation is always used initially in the default configuration. As you noticed, JDK-8282322 just makes it possible to create a build that uses the LSE variant from the start.<br>
<br>
JDK-8282322 requires an appropriate -march or similar option. The intrinsic<br>
approach gets the same desired effect (use only LSE) when such an option is<br>
provided, but inline.  That's one of the benefits.<br>
<br>
</blockquote></div>