[all-commits] [llvm/llvm-project] 68db7a: [OpenMP] Reorganize the initialization of `PluginA...

Johannes Doerfert via All-commits all-commits at lists.llvm.org
Tue Dec 5 16:04:15 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 68db7aef74c9992de2af69e2d0b127ea7c7eacc3
      https://github.com/llvm/llvm-project/commit/68db7aef74c9992de2af69e2d0b127ea7c7eacc3
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2023-12-05 (Tue, 05 Dec 2023)

  Changed paths:
    M openmp/libomptarget/include/PluginManager.h
    M openmp/libomptarget/include/device.h
    M openmp/libomptarget/plugins-nextgen/common/src/PluginInterface.cpp
    M openmp/libomptarget/src/PluginManager.cpp
    M openmp/libomptarget/src/device.cpp

  Log Message:
  -----------
  [OpenMP] Reorganize the initialization of `PluginAdaptorTy` (#74397)

This introduces checked errors into the creation and initialization of
`PluginAdaptorTy`. We also allow the adaptor to "hide" devices from the
user if the initialization failed. The new organization avoids the
"initOnce" stuff but we still do not eagerly initialize the plugin
devices (I think we should merge `PluginAdaptorTy::initDevices` into
`PluginAdaptorTy::init`)




More information about the All-commits mailing list