Re: [PATCH v4 12/24] PM / devfreq: tegra30: Inline all one-line functions

From: Dmitry Osipenko
Date: Tue Jul 16 2019 - 09:35:57 EST


16.07.2019 15:26, Chanwoo Choi ÐÐÑÐÑ:
> Hi Dmitry,
>
> I'm not sure that it is necessary.
> As I knew, usally, the 'inline' is used on header file
> to define the empty functions.
>
> Do we have to change it with 'inline' keyword?

The 'inline' attribute tells compiler that instead of jumping into the
function, it should take the function's code and replace the function's
invocation with that code. This is done in order to help compiler
optimize code properly, please see [1]. There is absolutely no need to
create a function call into a function that consists of a single
instruction.

[1] https://gcc.gnu.org/onlinedocs/gcc-9.1.0/gcc/Inline.html