Re: [PATCH 08/10] netfs: Use folio_next_pos()

[Matthew Wilcox <willy@infradead.org>]

On Mon, Oct 27, 2025 at 02:16:45PM +0000, David Howells wrote:
> Matthew Wilcox (Oracle) <willy@infradead.org> wrote:
> 
> > This is one instruction more efficient than open-coding folio_pos() +
> > folio_size().  It's the equivalent of (x + y) << z rather than
> > x << z + y << z.
> 
> Should that be noted to the gcc bugzilla as a missed optimisation?

I don't know?  Maybe there's a Law Of C Arithmetic that prevents it
from doing this optimisation that we can ignore because we know that
the calculation will never overflow.

I tried with both gcc and LLVM and both produce better code for g()
than for f():
https://godbolt.org/z/YTc883f6G

Interestingly, LLVM does better for f() than gcc does, but they produce
identical code for g() (minor difference like using sal instead of shl
but that feels unimportant).

If you want to file optimisation bug(s) against gcc, feel free to use
this example code.