Re: [PATCH] compiler: enable CONFIG_OPTIMIZE_INLINING forcibly
From: Linus Torvalds
Date: Thu Oct 03 2019 - 13:29:47 EST
On Thu, Oct 3, 2019 at 10:24 AM Masahiro Yamada
<yamada.masahiro@xxxxxxxxxxxxx> wrote:
>
> I just want to annotate __always_inline for the case
> "2. code that if not inlined is somehow not correct."
Oh, I support that entirely - if only for documentation.
But I do *not* support the dismissal of the architecture maintainers
concerns about "does it work?" and apparently known compiler bugs.
> Again, not saying "use a macro".
Other people did, though.
And there seemed to be little balancing of the pain vs the gain. The
gain really isn't that obvious. If the code shrinks by a couple of kB,
is that good or bad? Maybe it is smaller, but is it _better_?
Linus