Add end

Author: zhongmingyuan

devtools/etdump/etdump_flatcc.cpp

@@ -251,7 +251,7 @@ Result<EventTracerEntry> ETDumpGen::start_profiling_delegate(
   return Result<EventTracerEntry>(prof_entry);
 }
 
-void ETDumpGen::end_profiling_delegate(
+Result<bool> ETDumpGen::end_profiling_delegate(
     EventTracerEntry event_tracer_entry,
     const void* metadata,
     size_t metadata_len) {
@@ -282,6 +282,7 @@ void ETDumpGen::end_profiling_delegate(
   etdump_RunData_events_push_start(builder_);
   etdump_Event_profile_event_add(builder_, id);
   etdump_RunData_events_push_end(builder_);
+  return Result<bool>(true);
 }
 
 Result<bool> ETDumpGen::log_profiling_delegate(

devtools/etdump/etdump_flatcc.h

@@ -86,7 +86,7 @@ class ETDumpGen : public ::executorch::runtime::EventTracer {
   virtual Result<::executorch::runtime::EventTracerEntry> start_profiling_delegate(
       const char* name,
       DelegateDebugIntId delegate_debug_index) override;
-  virtual void end_profiling_delegate(
+  virtual Result<bool> end_profiling_delegate(
       ::executorch::runtime::EventTracerEntry prof_entry,
       const void* metadata,
       size_t metadata_len) override;

docs/source/delegate-debugging.md

@@ -100,7 +100,7 @@ To conclude an `EventTracerEntry`, `event_tracer_end_profiling_delegate` is simp
 Optionally, additional runtime `metadata` can also be logged at this point.
 
 ```c++
-void event_tracer_end_profiling_delegate(
+Result<bool> event_tracer_end_profiling_delegate(
     EventTracer* event_tracer,
     EventTracerEntry event_tracer_entry,
     const void* metadata = nullptr,

runtime/core/event_tracer.h

@@ -235,7 +235,7 @@ class EventTracer {
    * make it available for the user again in the post-processing stage.
    * @param[in] metadata_len Length of the metadata buffer.
    */
-  virtual void end_profiling_delegate(
+  virtual Result<bool> end_profiling_delegate(
       EventTracerEntry event_tracer_entry,
       const void* metadata = nullptr,
       size_t metadata_len = 0) = 0;

runtime/core/test/event_tracer_test.cpp

@@ -82,7 +82,7 @@ class DummyEventTracer : public EventTracer {
     return Result<EventTracerEntry>(EventTracerEntry());
   }
 
-  void end_profiling_delegate(
+  Result<bool> end_profiling_delegate(
       ET_UNUSED EventTracerEntry event_tracer_entry,
       ET_UNUSED const void* metadata,
       ET_UNUSED size_t metadata_len) override {