Formatting

Author: againull

source/adapters/level_zero/kernel.cpp

@@ -598,7 +598,8 @@ ur_result_t urKernelCreate(
 
   for (auto &Dev : Program->AssociatedDevices) {
     auto ZeDevice = Dev->ZeDevice;
-    // Program may be associated with all devices from the context but built only for subset of devices.
+    // Program may be associated with all devices from the context but built
+    // only for subset of devices.
     if (Program->getState(ZeDevice) != ur_program_handle_t_::state::Exe)
       continue;
 
@@ -633,7 +634,7 @@ ur_result_t urKernelCreate(
     for (auto ZeSubDevice : ZeSubDevices) {
       (*RetKernel)->ZeKernelMap[ZeSubDevice] = ZeKernel;
     }
-  } 
+  }
   // There is no any successfully built executable for program.
   if ((*RetKernel)->ZeKernelMap.empty())
     return UR_RESULT_ERROR_INVALID_PROGRAM_EXECUTABLE;

source/adapters/level_zero/program.cpp

@@ -122,9 +122,9 @@ ur_result_t urProgramCreateWithBinaryExp(
     ur_context_handle_t hContext, ///< [in] handle of the context instance
     uint32_t numDevices,          ///< [in] number of devices
     ur_device_handle_t
-        *phDevices, ///< [in][range(0, numDevices)] a pointer to a list of
-                    ///< device handles. The binaries are loaded for devices
-                    ///< specified in this list.
+        *phDevices,   ///< [in][range(0, numDevices)] a pointer to a list of
+                      ///< device handles. The binaries are loaded for devices
+                      ///< specified in this list.
     size_t *pLengths, ///< [in][range(0, numDevices)] array of sizes of program
                       ///< binaries specified by `pBinaries` (in bytes).
     const uint8_t *
@@ -472,8 +472,6 @@ ur_result_t urProgramLinkExp(
       }
     }
 
-    // TODO: Use the module of the first device as the interop module because of
-    // lack of multi-device support for interop case.
     ur_program_handle_t_ *UrProgram = new ur_program_handle_t_(hContext);
     *phProgram = reinterpret_cast<ur_program_handle_t>(UrProgram);
     for (uint32_t i = 0; i < numDevices; i++) {
@@ -488,12 +486,13 @@ ur_result_t urProgramLinkExp(
       // here. Clear values of the previous device first.
       BuildFlagPtrs.clear();
       for (uint32_t I = 0; I < count; I++) {
-        BuildFlagPtrs.push_back(phPrograms[I]->getBuildOptions(ZeDevice).c_str());
+        BuildFlagPtrs.push_back(
+            phPrograms[I]->getBuildOptions(ZeDevice).c_str());
       }
       ZeExtModuleDesc.pBuildFlags = BuildFlagPtrs.data();
       if (count == 1)
-          ZeModuleDesc.pBuildFlags = ZeExtModuleDesc.pBuildFlags[0];
-      
+        ZeModuleDesc.pBuildFlags = ZeExtModuleDesc.pBuildFlags[0];
+
       ze_result_t ZeResult =
           ZE_CALL_NOCHECK(zeModuleCreate, (ZeContext, ZeDevice, &ZeModuleDesc,
                                            &ZeModule, &ZeBuildLog));

source/adapters/level_zero/program.hpp

@@ -106,8 +106,8 @@ struct ur_program_handle_t_ : _ur_object {
   ur_program_handle_t_([[maybe_unused]] state St, ur_context_handle_t Context,
                        ze_module_handle_t InteropZeModule)
       : Context{Context}, NativeProperties{nullptr}, OwnZeModule{true},
-        AssociatedDevices({Context->getDevices()[0]}),
-        InteropZeModule{InteropZeModule} {}
+        AssociatedDevices({Context->getDevices()[0]}), InteropZeModule{
+                                                           InteropZeModule} {}
 
   // Construct a program in Exe state (interop).
   // TODO: Currently it is not possible to get the device associated with the
@@ -117,8 +117,8 @@ struct ur_program_handle_t_ : _ur_object {
   ur_program_handle_t_([[maybe_unused]] state St, ur_context_handle_t Context,
                        ze_module_handle_t InteropZeModule, bool OwnZeModule)
       : Context{Context}, NativeProperties{nullptr}, OwnZeModule{OwnZeModule},
-        AssociatedDevices({Context->getDevices()[0]}),
-        InteropZeModule{InteropZeModule} {
+        AssociatedDevices({Context->getDevices()[0]}), InteropZeModule{
+                                                           InteropZeModule} {
     // TODO: Currently it is not possible to understand the device associated
     // with provided ZeModule. So we can't set the state on that device to Exe.
   }