[Openmp-commits] [PATCH] D77822: [OpenMP] Put old APIs back and added new _async series for backward compatibility

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Apr 9 16:02:27 PDT 2020


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: openmp/libomptarget/src/omptarget.cpp:822
+
+  return OFFLOAD_SUCCESS;
 }
----------------
I'd prefer to always call sync and it's a noop if it should be. Oh, they are optional now. OK.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77822





More information about the Openmp-commits mailing list