[Openmp-commits] [PATCH] D108528: [OpenMP][Offloading] Add support for event related interfaces

Shilei Tian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Aug 27 09:53:24 PDT 2021

tianshilei1992 added a comment.

In D108528#2969435 <https://reviews.llvm.org/D108528#2969435>, @grokos wrote:

>> Changing plugin interfaces will be very painful. All changes since 2019 were all to add new interfaces just to make sure not to break existing applications.
> The interface you're talking about is between clang and libomptarget (`__tgt_target_*` functions). Here we are dealing with the interface between the base library and the plugins (`__tgt_rtl_*` functions), i.e. it is an internal interface within the library, so user applications have nothing to do with it. We can change it without breaking anything (I assume libomptarget and the plugins are compiled and distributed together, I don't expect anyone to mix-and-match components from different commits). The only caveat is that if the `__tgt_rtl_*` interface changes for one plugin, we have to propagate the change in all other plugins, but that shouldn't be a major concern.

Oh, yeah, you're right. However I can still remember back to 2019 when I added the `*_async` series, I did want to change existing interfaces, but it was rejected.

  rG LLVM Github Monorepo



More information about the Openmp-commits mailing list