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

[George Rokos via Phabricator via Openmp-commits]

grokos added a comment.

> 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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108528