Update on "[devtool] make datasink as a sepreate directory"

this diff make data_sink_base and its childrens as a seperate directory for better structure.

Differential Revision: [D69732404](https://our.internmc.facebook.com/intern/diff/D69732404/)

[ghstack-poisoned]

Author: Gasoonjia

devtools/CMakeLists.txt

@@ -176,8 +176,8 @@ add_custom_command(
 add_library(
   etdump ${CMAKE_CURRENT_SOURCE_DIR}/etdump/etdump_flatcc.cpp
          ${CMAKE_CURRENT_SOURCE_DIR}/etdump/emitter.cpp
-         ${CMAKE_CURRENT_SOURCE_DIR}/etdump/buffer_data_sink.cpp
-         ${CMAKE_CURRENT_SOURCE_DIR}/etdump/buffer_data_sink.h
+         ${CMAKE_CURRENT_SOURCE_DIR}/etdump/data_sinks/buffer_data_sink.cpp
+         ${CMAKE_CURRENT_SOURCE_DIR}/etdump/data_sinks/buffer_data_sink.h
 )
 
 target_link_libraries(

devtools/etdump/data_sinks/targets.bzl

@@ -1,6 +1,28 @@
 load("@fbsource//xplat/executorch/build:runtime_wrapper.bzl", "runtime")
 
 
+def define_data_sink_target(data_sink_name, aten_suffix):
+    runtime.cxx_library(
+            name = data_sink_name + aten_suffix,
+            exported_headers = [
+                data_sink_name + ".h",
+            ],
+            srcs = [
+                data_sink_name + ".cpp",
+            ],
+            deps = [
+                "//executorch/devtools/etdump:utils",
+            ],
+            exported_deps = [
+                "//executorch/runtime/core/exec_aten:lib" + aten_suffix,
+                ":data_sink_base" + aten_suffix,
+            ],
+            visibility = [
+                "//executorch/...",
+                "@EXECUTORCH_CLIENTS",
+            ],
+        )
+
 def define_common_targets():
     """Defines targets that should be shared between fbcode and xplat.
 
@@ -24,23 +46,4 @@ def define_common_targets():
             ],
         )
 
-        runtime.cxx_library(
-            name = "buffer_data_sink" + aten_suffix,
-            exported_headers = [
-                "buffer_data_sink.h",
-            ],
-            srcs = [
-                "buffer_data_sink.cpp",
-            ],
-            deps = [
-                "//executorch/devtools/etdump:utils",
-            ],
-            exported_deps = [
-                "//executorch/runtime/core/exec_aten:lib" + aten_suffix,
-                ":data_sink_base" + aten_suffix,
-            ],
-            visibility = [
-                "//executorch/...",
-                "@EXECUTORCH_CLIENTS",
-            ],
-        )
+        define_data_sink_target("buffer_data_sink", aten_suffix)