Apply suggestions from code review

Co-authored-by: Sergey Semenov <[email protected]>

Author: gzadica1

sycl/include/sycl/detail/common.hpp

@@ -144,7 +144,7 @@ class __SYCL_EXPORT tls_code_loc_t {
   /// entry has been set up, a default coe location is returned.
   const detail::code_location &query();
   /// @brief Returns true if the TLS slot was cleared when this object was
-  /// cunstructed.
+  /// constructed.
   bool isToplevel() const { return !MLocalScope; }
 
 private:

sycl/source/detail/queue_impl.cpp

@@ -415,7 +415,7 @@ event queue_impl::submitWithHandler(const std::shared_ptr<queue_impl> &Self,
         CGH.depends_on(DepEvents);
         HandlerFunc(CGH);
       },
-      Self, {}, true);
+      Self, /*CodeLoc*/ {}, /*IsTopCodeLoc*/ true);
 }
 
 template <typename HandlerFuncT, typename MemOpFuncT, typename... MemOpArgTs>

sycl/source/detail/scheduler/commands.cpp

@@ -1993,9 +1993,9 @@ void instrumentationAddExtraKernelMetadata(
     if (!SyclKernel->isCreatedFromSource())
       EliminatedArgMask = SyclKernel->getKernelArgMask();
   } else if (Queue) {
-    // NOTE: Queue can be null when kerner is directly enqueued to a command
+    // NOTE: Queue can be null when kernel is directly enqueued to a command
     // buffer
-    //       by graph API, when a midifiable graph is finalized.
+    //       by graph API, when a modifiable graph is finalized.
     std::tie(Kernel, KernelMutex, EliminatedArgMask, Program) =
         detail::ProgramManager::getInstance().getOrCreateKernel(
             Queue->getContextImplPtr(), Queue->getDeviceImplPtr(), KernelName);
@@ -2117,9 +2117,8 @@ std::pair<xpti_td *, uint64_t> emitKernelInstrumentationData(
 
   if (CmdTraceEvent) {
     // Stash the queue_id mutable metadata in TLS
-    // NOTE: Queue can be null when kerner is directly enqueued to a command
-    // buffer
-    //       by graph API, when a midifiable graph is finalized.
+    // NOTE: Queue can be null when kernel is directly enqueued to a command
+    // buffer by graph API, when a modifiable graph is finalized.
     if (Queue.get())
       xpti::framework::stash_tuple(XPTI_QUEUE_INSTANCE_ID_KEY,
                                    getQueueID(Queue));

sycl/unittests/xpti_trace/NodeCreation.cpp

@@ -167,8 +167,7 @@ TEST_F(NodeCreation, CommandGraphRecord) {
 
     auto exeGraph = cmdGraph.finalize();
 
-    // Notifications should have get generated during finalize
-    //
+    // Notifications should have been generated during finalize
     uint16_t TraceType = 0;
     std::string Message;
     ASSERT_TRUE(queryReceivedNotifications(TraceType, Message));