blob: 9254fe6d88e6da635a2a0f06a23315ed1b587cee [file] [log] [blame]
Benoit Jacob294f5f12009-01-12 14:41:12 +00001namespace Eigen {
2
Gael Guennebaud93ee82b2013-01-05 16:37:11 +01003/** \eigenManualPage TopicPassingByValue Passing Eigen objects by value to functions
Benoit Jacob294f5f12009-01-12 14:41:12 +00004
5Passing objects by value is almost always a very bad idea in C++, as this means useless copies, and one should pass them by reference instead.
6
Gael Guennebaud57acb052016-12-11 22:45:32 +01007With %Eigen, this is even more important: passing \ref TopicFixedSizeVectorizable "fixed-size vectorizable Eigen objects" by value is not only inefficient, it can be illegal or make your program crash! And the reason is that these %Eigen objects have alignment modifiers that aren't respected when they are passed by value.
Benoit Jacob294f5f12009-01-12 14:41:12 +00008
Gael Guennebaud57acb052016-12-11 22:45:32 +01009For example, a function like this, where \c v is passed by value:
Benoit Jacob294f5f12009-01-12 14:41:12 +000010
11\code
12void my_function(Eigen::Vector2d v);
13\endcode
14
Gael Guennebaud57acb052016-12-11 22:45:32 +010015needs to be rewritten as follows, passing \c v by const reference:
Benoit Jacob294f5f12009-01-12 14:41:12 +000016
17\code
18void my_function(const Eigen::Vector2d& v);
19\endcode
20
Gael Guennebaud57acb052016-12-11 22:45:32 +010021Likewise if you have a class having an %Eigen object as member:
Benoit Jacob294f5f12009-01-12 14:41:12 +000022
23\code
24struct Foo
25{
26 Eigen::Vector2d v;
27};
28void my_function(Foo v);
29\endcode
30
31This function also needs to be rewritten like this:
32\code
33void my_function(const Foo& v);
34\endcode
35
36Note that on the other hand, there is no problem with functions that return objects by value.
37
38*/
39
40}