[Openmp-commits] [PATCH] D148027: [OpenMP][libomptarget] Remove error data member from AsyncInfoWrapperTy

Kevin Sala via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Apr 11 09:52:00 PDT 2023


kevinsala created this revision.
kevinsala added reviewers: jdoerfert, ye-luo, jhuber6, tianshilei1992.
kevinsala added a project: OpenMP.
Herald added subscribers: sunshaoce, guansong, yaxunl.
Herald added a project: All.
kevinsala requested review of this revision.
Herald added subscribers: openmp-commits, jplehr, sstefan1.

This patch removes the `Err` data member from the AsyncInfoWrapperTy class. Now the error is stored externally, in the caller side, and it is explicitly passed to the `AsyncInfoWrapperTy::finalize()` function as a reference.

Proposed by @ye-luo in D142850 <https://reviews.llvm.org/D142850>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D148027

Files:
  openmp/libomptarget/plugins-nextgen/common/PluginInterface/PluginInterface.cpp
  openmp/libomptarget/plugins-nextgen/common/PluginInterface/PluginInterface.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148027.512499.patch
Type: text/x-patch
Size: 5931 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20230411/27a9cf93/attachment-0001.bin>


More information about the Openmp-commits mailing list