Re: [PATCH 3/3] make allocation fixed instead of variable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Tue, Feb 23, 2016 at 01:57:59PM -0600, Jonathon Jongsma wrote:
> > 
> > I'm a bit ambivalent about this change. Although it may generate better
> > code, I
> > think that it makes the code slightly more difficult to read. I'd almost
> > prefer
> > a #define to the enum solution. But I guess I don't care too much.
> 
> I definitely prefer a #define to the enum solution. But do we care at
> all about making these array sizes known at compile-time? Do we get any
> benefit from it?
> 
> Christophe
> 

I already discarded the patch.
Turned out that the missing optimization was due to a -O0 on my side.
I was trying to remove global variable (not const) from code and this
mis-optimization cause a variable to be added.
I updated my script to ignore these variables.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]