[Openmp-commits] [PATCH] D133053: [Libomptarget] Change device free routines to accept the allocation kind

Joseph Huber via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Sep 1 09:01:50 PDT 2022

jhuber6 added a comment.

In D133053#3764276 <https://reviews.llvm.org/D133053#3764276>, @tianshilei1992 wrote:

> In D133053#3764270 <https://reviews.llvm.org/D133053#3764270>, @jhuber6 wrote:
>> In D133053#3764266 <https://reviews.llvm.org/D133053#3764266>, @tianshilei1992 wrote:
>>> You can still use `_v2` or whatever in `libomptarget`. For `libomptarget`, there is no old interface then. Just keep the old one and all the logic for compatibility in the plugin. There is no confusion.
>> The potential change here is users who used `omp_target_free` to free non-device pointers as this worked before. We technically export the plugin routines as they need to be visible to `libomptarget` but I don't know if we expect any users to call them directly.
> The "user" here is old `libomptarget`.

It would be an incredibly weird setup to have `libomptarget` load a plugin from a different install right? We install them to the same directory so it would mean that the user manually edited `libomptarget` after the install.

  rG LLVM Github Monorepo



More information about the Openmp-commits mailing list