Commit 399bf8c9 authored by Boris Sazonov's avatar Boris Sazonov Committed by Chromium LUCI CQ

Revert "Reland "[TFLite] Enable tflite in build for android and desktop.""

This reverts commit 3f534050.

Reason for revert: breaks Android builders: https://crbug.com/1169083.

Original change's description:
> Reland "[TFLite] Enable tflite in build for android and desktop."
>
> This is a reland of a4df03d0
>
> This addresses the revert by removing neon from the TFLite build
> for chromeos.
>
>
> Original change's description:
> > [TFLite] Enable tflite in build for android and desktop.
> >
> > Bug: 1165517
> > Binary-Size: Expected increase due to inclusion of TFLite in binary.
> > Change-Id: I1c4c7b26f257c152d9477855ad32a77c0d30b85f
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616241
> > Reviewed-by: Andrew Grieve <agrieve@chromium.org>
> > Reviewed-by: Tarun Bansal <tbansal@chromium.org>
> > Reviewed-by: Sophie Chang <sophiechang@chromium.org>
> > Commit-Queue: Michael Crouse <mcrouse@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#845026}
>
> Cq-Include-Trybots: luci.chrome.try:chromeos-eve-chrome
> Bug: 1165517
> Binary-Size: Expected increase due to inclusion of TFLite in binary.
> TBR: Andrew Grieve <agrieve@chromium.org>
> TBR: Tarun Bansal <tbansal@chromium.org>
> Change-Id: I4b6b21b4a23a0c6978bfbb57150e9bf81300059f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2639648
> Commit-Queue: Michael Crouse <mcrouse@chromium.org>
> Reviewed-by: Tarun Bansal <tbansal@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#845478}

TBR=agrieve@chromium.org,tbansal@chromium.org,sophiechang@chromium.org,mcrouse@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com

Change-Id: I7475d86a39c33285c37662fa8a1c7f44f4229e1c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1165517
Cq-Include-Trybots: luci.chrome.try:chromeos-eve-chrome
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2642619Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845612}
parent f7288764
...@@ -4,6 +4,5 @@ ...@@ -4,6 +4,5 @@
declare_args() { declare_args() {
# This enables build with TFLite library. # This enables build with TFLite library.
# Currently only available for Desktop and Android. build_with_tflite_lib = false
build_with_tflite_lib = is_android || is_win || is_linux || is_mac || is_chromeos
} }
...@@ -37,12 +37,6 @@ config("tflite_flags") { ...@@ -37,12 +37,6 @@ config("tflite_flags") {
if (is_linux || is_chromeos_lacros) { if (is_linux || is_chromeos_lacros) {
defines += [ "GEMMLOWP_ALLOW_SLOW_SCALAR_FALLBACK" ] defines += [ "GEMMLOWP_ALLOW_SLOW_SCALAR_FALLBACK" ]
} }
if (is_chromeos) {
# TODO(crbug.com/1168787): Determine if neon can be supported
# on chromeos builds.
defines += ["TF_LITE_DISABLE_X86_NEON"]
}
} }
source_set("tflite_public_headers") { source_set("tflite_public_headers") {
...@@ -59,7 +53,7 @@ source_set("tflite_public_headers") { ...@@ -59,7 +53,7 @@ source_set("tflite_public_headers") {
"src/tensorflow/lite/graph_info.h", "src/tensorflow/lite/graph_info.h",
"src/tensorflow/lite/interpreter.h", "src/tensorflow/lite/interpreter.h",
"src/tensorflow/lite/interpreter_builder.h", "src/tensorflow/lite/interpreter_builder.h",
"src/tensorflow/lite/kernels/builtin_op_kernels.h", "src/tensorflow/lite/kernls/builtin_op_kernels.h",
"src/tensorflow/lite/model.h", "src/tensorflow/lite/model.h",
"src/tensorflow/lite/model_builder.h", "src/tensorflow/lite/model_builder.h",
"src/tensorflow/lite/mutable_op_resolver.h", "src/tensorflow/lite/mutable_op_resolver.h",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment