Update

[ghstack-poisoned]

Author: swolchok

CMakeLists.txt

@@ -548,6 +548,16 @@ if(EXECUTORCH_BUILD_EXTENSION_DATA_LOADER)
   add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/extension/data_loader)
 endif()
 
+if(EXECUTORCH_BUILD_EXTENSION_EVALUE_UTIL)
+  add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/extension/evalue_util)
+  install(
+    DIRECTORY extension/evalue_util/
+    DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/executorch/extension/evalue_util
+    FILES_MATCHING
+    PATTERN "*.h"
+  )
+endif()
+
 if(EXECUTORCH_BUILD_EXTENSION_FLAT_TENSOR)
   add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/extension/flat_tensor)
 endif()
@@ -576,6 +586,12 @@ endif()
 
 if(EXECUTORCH_BUILD_EXTENSION_RUNNER_UTIL)
   add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/extension/runner_util)
+  install(
+    DIRECTORY extension/runner_util/
+    DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/executorch/extension/runner_util
+    FILES_MATCHING
+    PATTERN "*.h"
+  )
 endif()
 
 if(EXECUTORCH_BUILD_EXTENSION_TENSOR)
@@ -651,8 +667,7 @@ if(EXECUTORCH_BUILD_PYBIND)
 
   # util lib
   add_library(
-    util ${CMAKE_CURRENT_SOURCE_DIR}/extension/evalue_util/print_evalue.cpp
-         ${CMAKE_CURRENT_SOURCE_DIR}/extension/aten_util/aten_bridge.cpp
+    util ${CMAKE_CURRENT_SOURCE_DIR}/extension/aten_util/aten_bridge.cpp
   )
   target_include_directories(
     util PUBLIC ${_common_include_directories} ${TORCH_INCLUDE_DIRS}
@@ -695,7 +710,9 @@ endif()
 
 if(EXECUTORCH_BUILD_EXECUTOR_RUNNER)
   # Baseline libraries that executor_runner will link against.
-  set(_executor_runner_libs executorch gflags)
+  set(_executor_runner_libs executorch extension_evalue_util
+                            extension_runner_util gflags
+  )
 
   if(EXECUTORCH_BUILD_KERNELS_OPTIMIZED)
     list(APPEND _executor_runner_libs optimized_native_cpu_ops_lib)

examples/apple/mps/CMakeLists.txt

@@ -105,6 +105,8 @@ if(NOT CMAKE_TOOLCHAIN_FILE MATCHES ".*(iOS|ios\.toolchain)\.cmake$")
     mps_executor_runner
     bundled_program
     executorch
+    extension_evalue_util
+    extension_runner_util
     gflags
     etdump
     flatccrt

examples/apple/mps/test_mps.sh

@@ -19,6 +19,8 @@ cmake_install_executorch_devtools_lib() {
           -DCMAKE_INSTALL_PREFIX=cmake-out \
           -DCMAKE_BUILD_TYPE=Release \
           -DEXECUTORCH_BUILD_DEVTOOLS=ON \
+          -DEXECUTORCH_BUILD_EXTENSION_EVALUE_UTIL=ON \
+          -DEXECUTORCH_BUILD_EXTENSION_RUNNER_UTIL=ON \
           -DEXECUTORCH_BUILD_MPS=ON \
           -DEXECUTORCH_ENABLE_EVENT_TRACER=ON \
           -DPYTHON_EXECUTABLE="$PYTHON_EXECUTABLE" \

examples/qualcomm/executor_runner/CMakeLists.txt

@@ -19,8 +19,14 @@ target_include_directories(
   qnn_executor_runner PUBLIC ${_common_include_directories}
 )
 target_link_libraries(
-  qnn_executor_runner qnn_executorch_backend full_portable_ops_lib etdump
-  flatccrt gflags
+  qnn_executor_runner
+  qnn_executorch_backend
+  extension_evalue_util
+  extension_runner_util
+  full_portable_ops_lib
+  etdump
+  flatccrt
+  gflags
 )
 set_target_properties(
   qnn_executor_runner PROPERTIES LINK_FLAGS "-Wl,-rpath='$ORIGIN'"

extension/evalue_util/CMakeLists.txt

@@ -0,0 +1,29 @@
+# Copyright (c) Meta Platforms, Inc. and affiliates.
+# All rights reserved.
+#
+# This source code is licensed under the BSD-style license found in the
+# LICENSE file in the root directory of this source tree.
+
+# Please this file formatted by running:
+# ~~~
+# cmake-format -i CMakeLists.txt
+# ~~~
+# Source root directory for executorch.
+if(NOT EXECUTORCH_ROOT)
+  set(EXECUTORCH_ROOT ${CMAKE_CURRENT_SOURCE_DIR}/../..)
+endif()
+list(TRANSFORM _extension_evalue_util__srcs PREPEND "${EXECUTORCH_ROOT}/")
+add_library(extension_evalue_util ${_extension_evalue_util__srcs})
+target_link_libraries(extension_evalue_util executorch_core)
+target_include_directories(
+  extension_evalue_util PUBLIC ${_common_include_directories}
+)
+target_compile_options(extension_evalue_util PUBLIC ${_common_compile_options})
+
+# Install libraries
+install(
+  TARGETS extension_evalue_util
+  DESTINATION ${CMAKE_BINARY_DIR}/lib
+  INCLUDES
+  DESTINATION ${_common_include_directories}
+)

tools/cmake/cmake_deps.toml

@@ -198,6 +198,17 @@ deps = [
   "executorch_core",
 ]
 
+[targets.extension_evalue_util]
+buck_targets = [
+  "//extension/evalue_util:print_evalue",
+]
+filters = [
+  ".cpp$",
+]
+deps = [
+  "executorch_core",
+]
+
 [targets.extension_flat_tensor_schema]
 buck_targets = [
   "//extension/flat_tensor/serialize:generated_headers",
@@ -340,6 +351,8 @@ excludes = [
 deps = [
   "executorch",
   "executorch_core",
+  "extension_evalue_util",
+  "extension_runner_util",
   "extension_threadpool",
   "kernels_util_all_deps",
   "portable_kernels",
@@ -376,6 +389,8 @@ excludes = [
 deps = [
   "executorch",
   "executorch_core",
+  "extension_evalue_util",
+  "extension_runner_util",
   "extension_threadpool",
   "kernels_util_all_deps",
   "portable_kernels",
@@ -417,6 +432,8 @@ excludes = [
 deps = [
   "executorch",
   "executorch_core",
+  "extension_evalue_util",
+  "extension_runner_util",
   "extension_threadpool",
   "kernels_util_all_deps",
   "xnnpack_backend",

tools/cmake/preset/default.cmake

@@ -82,10 +82,6 @@ define_overridable_option(
 define_overridable_option(
   EXECUTORCH_BUILD_EXTENSION_MODULE "Build the Module extension" BOOL OFF
 )
-define_overridable_option(
-  EXECUTORCH_BUILD_EXTENSION_RUNNER_UTIL "Build the Runner Util extension" BOOL
-  OFF
-)
 define_overridable_option(
   EXECUTORCH_BUILD_EXTENSION_TENSOR "Build the Tensor extension" BOOL OFF
 )
@@ -177,6 +173,14 @@ define_overridable_option(
   EXECUTORCH_BUILD_EXECUTOR_RUNNER "Build the executor_runner executable" BOOL
   ${_default_executorch_build_executor_runner}
 )
+define_overridable_option(
+  EXECUTORCH_BUILD_EXTENSION_EVALUE_UTIL "Build the EValue util extension" BOOL
+  ${_default_executorch_build_executor_runner}
+)
+define_overridable_option(
+  EXECUTORCH_BUILD_EXTENSION_RUNNER_UTIL "Build the Runner Util extension" BOOL
+  ${_default_executorch_build_executor_runner}
+)
 
 # NB: Enabling this will serialize execution of delegate instances Keeping this
 # OFF by default to maintain existing behavior, to be revisited.
@@ -213,6 +217,11 @@ check_required_options_on(
   IF_ON EXECUTORCH_ENABLE_EVENT_TRACER REQUIRES EXECUTORCH_BUILD_DEVTOOLS
 )
 
+check_required_options_on(
+  IF_ON EXECUTORCH_BUILD_EXECUTOR_RUNNER REQUIRES
+  EXECUTORCH_BUILD_EXTENSION_EVALUE_UTIL
+  EXECUTORCH_BUILD_EXTENSION_RUNNER_UTIL
+)
 check_required_options_on(
   IF_ON EXECUTORCH_BUILD_EXTENSION_FLAT_TENSOR REQUIRES
   EXECUTORCH_BUILD_EXTENSION_DATA_LOADER