[Openmp-commits] [PATCH] D109341: [OpenMP][libomptarget] Change synchronize_ty return type to int32_t

Ye Luo via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Sep 6 19:34:54 PDT 2021

ye-luo created this revision.
Herald added subscribers: guansong, yaxunl.
ye-luo requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: openmp-commits, sstefan1.
Herald added a project: OpenMP.

Plugins always return int32_t. Stay consistent with other functions which return error status.

  rG LLVM Github Monorepo



Index: openmp/libomptarget/src/rtl.h
--- openmp/libomptarget/src/rtl.h
+++ openmp/libomptarget/src/rtl.h
@@ -52,7 +52,7 @@
                                             int32_t, uint64_t,
                                             __tgt_async_info *);
   typedef int64_t(init_requires_ty)(int64_t);
-  typedef int64_t(synchronize_ty)(int32_t, __tgt_async_info *);
+  typedef int32_t(synchronize_ty)(int32_t, __tgt_async_info *);
   typedef int32_t (*register_lib_ty)(__tgt_bin_desc *);
   typedef int32_t(supports_empty_images_ty)();
   typedef void(print_device_info_ty)(int32_t);

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109341.370984.patch
Type: text/x-patch
Size: 651 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20210907/c2ed7777/attachment.bin>

More information about the Openmp-commits mailing list