[Openmp-commits] [PATCH] D134396: [OpenMP][libomptarget] New plugin infrastructure and new CUDA plugin

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Oct 13 16:53:11 PDT 2022


jdoerfert added a comment.

@jhuber6 @tianshilei1992 Can we merge this and address changes in-tree?



================
Comment at: openmp/libomptarget/include/Utilities.h:57
+  std::string consumeString() { return toString(std::move(*this)); }
+};
+
----------------
Moving *this, couldn't that invalidate *this? If we know this is save, add a comment about it.


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

https://reviews.llvm.org/D134396



More information about the Openmp-commits mailing list