Re: [eigen] Array, Matrix, PlainObjectBase and DenseStorage are now move enabled

[Hauke Heibel <hauke.heibel@xxxxxxxxx>]

Hi Luke,

On Thu, Aug 8, 2013 at 3:01 AM, Dale Lukas Peterson <hazelnusse@xxxxxxxxx> wrote:

Basically, it says you should check for c++11 support via:
#if __cplusplus >= 201103L
....
#endif

As opposed to:
#ifdef __GXX_EXPERIMENTAL_CXX0X__
....
#endif

 The issue here is incomplete C++11 support of different compilers. We are already using this check in Eigen but MathFunctions.h in line 341 but I had to change the test to

#if (__cplusplus >= 201103L) && !defined(__CYGWIN__)

because on Cygwin std::log1p does not yet exists. So we need to do those checks feature by feature and compiler by compiler. 

For the move support it will probably work and I already recognized that __GXX_EXPERIMENTAL_CXX0X__ is not a very good check. I think I will try to add the flag you suggested and see what happens.

Regards,

Hauke