<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 14, 2018 at 12:48 AM Stephan Bergmann <<a href="mailto:sbergman@redhat.com">sbergman@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 13/09/2018 18:22, David Blaikie via llvm-dev wrote:<br>
> On Thu, Sep 13, 2018 at 12:13 AM Dávid Bolvanský via llvm-dev <br>
> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a> <mailto:<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>> wrote:<br>
>     /home/davidbolvansky/trunk/llvm/unittests/ExecutionEngine/Orc/CompileOnDemandLayerTest.cpp:79:40: <br>
>       required from here<br>
>     /home/davidbolvansky/trunk/llvm/include/llvm/ExecutionEngine/Orc/CompileOnDemandLayer.h:314:29:<br>
>     warning: redundant move in return statement [-Wredundant-move]<br>
>     314 |         return std::move(Err);<br>
<br>
Note that the move (into the implicit JITSymbol(Error) ctor) is only <br>
redundant if the compiler implements a fix for <br>
<<a href="http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#1579" rel="noreferrer" target="_blank">http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#1579</a>> <br>
"Return by converting move constructor".  (But not sure whether the LLVM <br>
compiler baselines imply that, anyway. </blockquote><div><br>Looks like that was accepted for C++14, not 11. So I don't believe it's valid to remove the std::move in C++11 code. (& I believe Clang's warning correctly diagnoses this only in the right language versions) - so we probably want to disable the buggy GCC warning here, then.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> In LibreOffice it forced me to <br>
introduce ugly #ifs, to not have to disable that warning outright, <br>
<<a href="https://cgit.freedesktop.org/libreoffice/core/commit/?id=dc06c8f4989fc28d0c31ebd333e53dfe0e0f5f66" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/libreoffice/core/commit/?id=dc06c8f4989fc28d0c31ebd333e53dfe0e0f5f66</a>> <br>
"-Werror=redundant-move (GCC 9), take two".)<br>
</blockquote></div></div>