Fix useless move() in IRCompileLayer.h

[emaxx]

Summary

Drop std::move() calls in places where there's no actual moving
happing, and only the "bugprone-use-after-move" clang-tidy
warning is triggered.

[emaxx]

@lhames - I uploaded this change before seeing your reply on GitHub: https://github.com/llvm/llvm-project/pull/250 . I'm fine with abandoning this if you're saying this will be fixed anyway - just let me know. Thanks.

[lhames]

Hi @emaxx,

Yes, please go ahead and abandon this review -- it will be fixed as soon as the orcv1-removal patch lands. Thank you for the contribution though!

Regards,
Lang.

[emaxx]

Abandoning as per comment from @lhames.

llvm/include/llvm/ExecutionEngine/Orc/IRCompileLayer.h

     auto Obj = Compile(*M);
     if (!Obj)
       return Obj.takeError();
-    if (auto Err = BaseLayer.addObject(std::move(K), std::move(*Obj)))
+    if (auto Err = BaseLayer.addObject(K, std::move(*Obj)))
       return Err;
     if (NotifyCompiled)
-      NotifyCompiled(std::move(K), std::move(M));
+      NotifyCompiled(K, std::move(M));
     return Error::success();
   }