[Openmp-commits] [openmp] r341399 - [libomptarget] Remove `Devices` from `RTLInfoTy`

[Sergey Dmitriev via Openmp-commits]

Author: sdmitriev
Date: Tue Sep  4 13:23:09 2018
New Revision: 341399

URL: http://llvm.org/viewvc/llvm-project?rev=341399&view=rev
Log:
[libomptarget] Remove `Devices` from `RTLInfoTy`

This patch removes unused field `Devices` from `RTLInfoTy`.

Differential Revision: https://reviews.llvm.org/D51653

Modified:
    openmp/trunk/libomptarget/src/rtl.cpp
    openmp/trunk/libomptarget/src/rtl.h

Modified: openmp/trunk/libomptarget/src/rtl.cpp
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/libomptarget/src/rtl.cpp?rev=341399&r1=341398&r2=341399&view=diff
==============================================================================
--- openmp/trunk/libomptarget/src/rtl.cpp (original)
+++ openmp/trunk/libomptarget/src/rtl.cpp Tue Sep  4 13:23:09 2018
@@ -223,9 +223,6 @@ void RTLsTy::RegisterLib(__tgt_bin_desc
           Devices[start + device_id].DeviceID = start + device_id;
           // RTL local device ID
           Devices[start + device_id].RTLDeviceID = device_id;
-
-          // Save pointer to device in RTL in case we want to unregister the RTL
-          R.Devices.push_back(&Devices[start + device_id]);
         }
 
         // Initialize the index of this RTL and save it in the used RTLs.

Modified: openmp/trunk/libomptarget/src/rtl.h
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/libomptarget/src/rtl.h?rev=341399&r1=341398&r2=341399&view=diff
==============================================================================
--- openmp/trunk/libomptarget/src/rtl.h (original)
+++ openmp/trunk/libomptarget/src/rtl.h Tue Sep  4 13:23:09 2018
@@ -43,7 +43,6 @@ struct RTLInfoTy {
                                    // i.e. the OpenMP index of the first device
                                    // to be registered with this RTL.
   int32_t NumberOfDevices;         // Number of devices this RTL deals with.
-  std::vector<DeviceTy *> Devices; // one per device (NumberOfDevices in total).
 
   void *LibraryHandler;
 
@@ -74,7 +73,7 @@ struct RTLInfoTy {
   // The existence of the mutex above makes RTLInfoTy non-copyable.
   // We need to provide a copy constructor explicitly.
   RTLInfoTy()
-      : Idx(-1), NumberOfDevices(-1), Devices(), LibraryHandler(0),
+      : Idx(-1), NumberOfDevices(-1), LibraryHandler(0),
 #ifdef OMPTARGET_DEBUG
         RTLName(),
 #endif
@@ -86,7 +85,6 @@ struct RTLInfoTy {
   RTLInfoTy(const RTLInfoTy &r) : Mtx() {
     Idx = r.Idx;
     NumberOfDevices = r.NumberOfDevices;
-    Devices = r.Devices;
     LibraryHandler = r.LibraryHandler;
 #ifdef OMPTARGET_DEBUG
     RTLName = r.RTLName;