Re: [eigen] patch (enums)

[ Thread Index | Date Index | More Archives ]

--- Benoît Jacob <jacob@xxxxxxxxxxxxxxx> wrote:

> Hi List,
> please find attached the patch for removing the "static const int" everywhere. 
> Instead, using enums in classes and #defines for the global constants (just 
> to test performance).
> Here, it does not at all change performance, both with g++-4.2 and g++-4.1!
> Therefore I am not so sure anymore if I want to move to enums. Please share 
> your opinion!
> It seems to me that g++ had no problem at all optimizing the "static const 
> int" as mere literals!

I think that the static const ints are unnecessarily stored in data segment of the compiled
program. It would be good to check the size difference of the executables before you revert back
to static const ints.


Mail converted by MHonArc 2.6.19+