Skip to content

Commit af098c3

Browse files
digantdesaifacebook-github-bot
authored andcommitted
Enable Workspace sharing by default (#5336)
Summary: Pull Request resolved: #5336 as Title bypass-github-export-checks bypass-github-pytorch-ci-checks bypass-github-executorch-ci-checks Reviewed By: kimishpatel, GregoryComer Differential Revision: D62598508 fbshipit-source-id: 6d2e6c36adb6365ba55c50fc99ca8a4562280a50
1 parent 90d5191 commit af098c3

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

backends/xnnpack/CMakeLists.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@ if(NOT PYTHON_EXECUTABLE)
3232
resolve_python_executable()
3333
endif()
3434

35-
# NB: Enabling this will serialize execution of delegate instances Keeping this
36-
# OFF by default to maintain existing behavior, to be revisited.
35+
# NB: Enabling this will serialize execution of delegate instances.
36+
# This setting may have performance implications.
3737
option(EXECUTORCH_XNNPACK_SHARED_WORKSPACE
38-
"Enable workspace sharing across different delegate instances" OFF
38+
"Enable workspace sharing across different delegate instances" ON
3939
)
4040
if(EXECUTORCH_XNNPACK_SHARED_WORKSPACE)
4141
add_definitions(-DENABLE_XNNPACK_SHARED_WORKSPACE)

backends/xnnpack/targets.bzl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,10 @@ def define_common_targets():
3636
"@EXECUTORCH_CLIENTS",
3737
],
3838
preprocessor_flags = [
39+
# Enable workspace sharing across delegates
40+
"-DENABLE_XNNPACK_SHARED_WORKSPACE",
3941
# Uncomment to enable per operator timings
4042
# "-DENABLE_XNNPACK_PROFILING",
41-
# Uncomment to enable workspace sharing across delegates
42-
# "-DENABLE_XNNPACK_SHARED_WORKSPACE"
4343
],
4444
exported_deps = [
4545
"//executorch/runtime/backend:interface",

0 commit comments

Comments
 (0)