[PATCH] D147544: [BOLT] Move from RuntimeDyld to JITLink

[Job Noorman via Phabricator via llvm-commits]

jobnoorman marked an inline comment as done.
jobnoorman added inline comments.


================
Comment at: bolt/lib/Rewrite/ExecutableFileMemoryManager.cpp:34
+  });
+  return std::move(Sections);
+}
----------------
rafauler wrote:
> Some warnings from clang here:
> 
> ExecutableFileMemoryManager.cpp:34:10: warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
>   return std::move(Sections);
>          ^
> ExecutableFileMemoryManager.cpp:34:10: note: remove std::move call here
>   return std::move(Sections);
Woops, darn you GCC :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147544/new/

https://reviews.llvm.org/D147544