Skip to content

Commit

Permalink
lite: Update Windows tensorflowlite_flex.dll build
Browse files Browse the repository at this point in the history
Removed "windows_export_all_symbols" feature since Flex delegate only requires
to expose `TF_AcquireFlexDelegate` symbol.
This change is needed for Issue#43367.

PiperOrigin-RevId: 424060071
Change-Id: I8874ce6b107f6db9c5445b65e55073ea46266c76
  • Loading branch information
terryheo committed Jan 27, 2022
1 parent 2962417 commit 8dd07bd
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion tensorflow/lite/delegates/flex/BUILD
Expand Up @@ -126,7 +126,7 @@ tflite_flex_cc_library(
# ops and kernels. The output library name is platform dependent:
# - Linux/Android: `libtensorflowlite_flex.so`
# - Mac: `libtensorflowlite_flex.dylib`
# - Windows: `libtensorflowlite_flex.dll`
# - Windows: `tensorflowlite_flex.dll`
tflite_flex_shared_library(
name = "tensorflowlite_flex",
)
Expand Down
3 changes: 0 additions & 3 deletions tensorflow/lite/delegates/flex/build_def.bzl
Expand Up @@ -216,9 +216,6 @@ def tflite_flex_shared_library(

tflite_cc_shared_object(
name = name,
# Until we have more granular symbol export for the C++ API on Windows,
# export all symbols.
features = ["windows_export_all_symbols"],
linkopts = select({
"//tensorflow:macos": [
"-Wl,-exported_symbols_list,$(location //tensorflow/lite/delegates/flex:exported_symbols.lds)",
Expand Down

0 comments on commit 8dd07bd

Please sign in to comment.