Re: [Intel-gfx] [PATCH v2 09/11] drm: Convert open-coded yes/no strings to yesno()

From: Andy Shevchenko
Date: Wed Jan 26 2022 - 07:16:28 EST


On Wed, Jan 26, 2022 at 02:43:45AM -0800, Lucas De Marchi wrote:
> On Wed, Jan 26, 2022 at 12:12:50PM +0200, Andy Shevchenko wrote:
> > On Wed, Jan 26, 2022 at 11:39 AM Lucas De Marchi
> > <lucas.demarchi@xxxxxxxxx> wrote:

...

> > > 411986 10490 6176 428652 68a6c drm.ko.old
> > > 411986 10490 6176 428652 68a6c drm.ko
> > > 98129 1636 264 100029 186bd dp/drm_dp_helper.ko.old
> > > 98129 1636 264 100029 186bd dp/drm_dp_helper.ko
> > > 1973432 109640 2352 2085424 1fd230 nouveau/nouveau.ko.old
> > > 1973432 109640 2352 2085424 1fd230 nouveau/nouveau.ko
> >
> > This probably won't change for modules, but if you compile in the
> > linker may try to optimize it. Would be nice to see the old-new for
> > `make allyesconfig` or equivalent.
>
> just like it would already do, no? I can try and see what happens, but
> my feeling is that we won't have any change.

Maybe not or maybe a small win. Depends how compiler puts / linker sees
that in two cases. (Yeah, likely it should be no differences if all
instances are already caught by linker)

--
With Best Regards,
Andy Shevchenko