Re: [Qemu-devel] [RFC PATCH 0/6] Convert softmmu-template into normal code

[Emilio G. Cota]

On Fri, Apr 20, 2018 at 16:11:35 -0300, Philippe Mathieu-Daudé wrote:
> On 04/20/2018 03:35 PM, Emilio G. Cota wrote:
> > I also changed the patches to make the series simpler: one patch
> > to add softmmu.c (without compiling it yet), one patch to
> > actually use softmmu.c, and a final patch to remove the template
> > file.
> 
> By "without compiling" you mean it is not compilable?
> That would mean not script-bisectable.

I mean it's committed but not yet buildable, i.e. makefiles
don't know anything about it. (So no, the build isn't broken.)

The point is to make the diff's as easy to review as possible.

> > The executable is now larger:
> 
> This is odd, I was expecting some shrinking...
>
> What kind of build are doing? More such -ggdb or such -O3?

I'm using the defaults.

Why would you expect a size reduction? With flatten we tell the
compiler to expand everything it can, so it makes sense that
it leads to larger code size vs. letting the compiler decide
what (or not) to inline.

Cheers,

                Emilio