[all-commits] [llvm/llvm-project] 2cad43: [Libomptarget] Factor functions out of 'Plugin' in...

[Joseph Huber via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2cad43c1ba7d9c83ae8fb809e60a57d347e09370
      https://github.com/llvm/llvm-project/commit/2cad43c1ba7d9c83ae8fb809e60a57d347e09370
  Author: Joseph Huber <huberjn at outlook.com>
  Date:   2024-03-25 (Mon, 25 Mar 2024)

  Changed paths:
    M openmp/libomptarget/CMakeLists.txt
    M openmp/libomptarget/plugins-nextgen/amdgpu/src/rtl.cpp
    M openmp/libomptarget/plugins-nextgen/common/include/PluginInterface.h
    M openmp/libomptarget/plugins-nextgen/common/src/PluginInterface.cpp
    M openmp/libomptarget/plugins-nextgen/cuda/src/rtl.cpp
    M openmp/libomptarget/plugins-nextgen/host/src/rtl.cpp

  Log Message:
  -----------
  [Libomptarget] Factor functions out of 'Plugin' interface (#86528)

Summary:
This patch factors common functions out of the `Plugin` interface prior
to its removal in a future patch. This simply temporarily renames it to
`PluginTy` so that we could re-use `Plugin::check` internally as this
needs to be defined statically per plugin now. We can refactor this
later.

The future patch will delete `PluginTy` and `PluginTy::get` entirely.
This simply tries to minimize a few changes to make it easier to land.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications