Sollte ich hier das move weglassen, weil der Compiler s selbst als Temporary erkennt?
Es hieß mal irgendwo, dass

return move(value);

ineffizienter als

return value;

bei lokalen Variablen ist.

Sollte ich hier das move weglassen, weil der Compiler s selbst als Temporary erkennt?

Selbst wenn der Compiler so schlau sein sollte, dass er das erkennt, darf er hier kein move automatisch verwenden -- zumindest nicht in einer "as-if"-Regel-verletzenden Weise. Deine Verwendung von std::move ist hier also genau richtig und vordbildlich.

Die einzige Stelle, bei der ein Compiler ein Lvalue als Rvalue behandeln darf, ist beim return eines funktionslokalen Objekts.

nwp3 schrieb:

Es hieß mal irgendwo, dass

return move(value);

ineffizienter als

return value;

bei lokalen Variablen ist.

Genau. Du hebelst nämlich durch diese Verwendung von move die NRVO-Optimierung aus, die ein Compiler hier sonst anwenden dürfte (und typischerweise auch tut). NRVO ist noch günstiger als eine Move-Konstruktion. Mit std::move würdest du hier mindestens eine Move-Konstruktion bekommen, weil die Copy-Elision-Regel nicht mehr greifen kann. Das liegt daran, dass std::move etwas mehr als ein Cast ist: Es ist ein Funktionsaufruf -- auch wenn dieser geinlined werden kann und die Funktion sonst nichts besonderes macht.

...

Vielen Dank!