Re: [PATCH]cgroup: __cpuset_node_allowed return bool

From: Tejun Heo
Date: Mon Mar 26 2018 - 10:25:36 EST


On Mon, Mar 26, 2018 at 10:20:43PM +0800, yuankuiz@xxxxxxxxxxxxxx wrote:
> 1) return int type variable in bool function:
> bool enabled()
> {
> int ret = 1;
> return ret;
> }
...
> 2)
> bool enabled()
> {
> bool ret = 1;
> return ret;
> }
...
> so the #1) style function can generate significant instructions
> than the #2).

That is a problem for the compiler, not the code.

> While, this is happened only when "-On" is not used with *-gcc
> together. Though, it is oftern there, it is best to provide this
> with decoupling of which option is used for optimization.

We don't want to dictate minute coding styles to avoid things which
are trivially optimized by compilers.

Thanks.

--
tejun