On Fri, 21 Aug 2026 00:04:36 GMT, Shawn Emery <[email protected]> wrote:

> > This is starting to look good. A much-needed cleanup!
> > One more thing I'd do, while I was at it. There are several instances of 
> > this pattern:
> > ```
> >       if (!can_use_fp || !can_use_r18) {
> >         __ ldp(tmp3, tmp4, __ post(sp, 16));
> >       }
> > ```
> > 
> >   
> > which can be replaced by unconditional `push` and `pop`.
> 
> I'm concerned that the unconditional `push` & `pop` here, in that it could 
> have some performance impact for Linux-Neoverse systems. As this would create 
> artificial spillage whereas none was needed before this change.

Can you show us an example of this?

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

PR Comment: https://git.openjdk.org/jdk/pull/32049#issuecomment-5369278943

Reply via email to