List array.cc in the MICROLITE_CC_BASE_SRC

[kraiskil]

Hello,

I'm following the steps in tensorflow/lite/micro/docs/new_platform_support.md document.

I'm compiling the hello_world example with a CMakeLists.txt posted below. It fails with

tflm_project/tensorflow/lite/kernels/kernel_util.cc:28:10: fatal error: tensorflow/lite/array.h: No such file or directory
   28 | #include "tensorflow/lite/array.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

With the the proposed changes, the build succeeds. This would be my first contribution to tensorflow. Does it make sense? I don't really know what else this change affects.

The diff is a bit ugly because I kept the alphabetical order in the file lists - should this be modified?

kalle

---

CMakeListst.txt to compile the generated hello_world.

cmake_minimum_required(VERSION 2.8.12)
project( TFLM
    LANGUAGES CXX )
file( GLOB_RECURSE SRCS 
    examples/hello_world/*.cc
    signal/*.cc 
    tensorflow/*.cc
    third_party/*.cc
    )
add_executable( hello_world ${SRCS}) 
target_include_directories( hello_world 
    PUBLIC
        ${CMAKE_CURRENT_SOURCE_DIR}/ 
        ${CMAKE_CURRENT_SOURCE_DIR}/examples/hello_world
        ${CMAKE_CURRENT_SOURCE_DIR}/third_party/flatbuffers/include
        ${CMAKE_CURRENT_SOURCE_DIR}/third_party/gemmlowp
        ${CMAKE_CURRENT_SOURCE_DIR}/third_party/ruy
        ${CMAKE_CURRENT_SOURCE_DIR}/third_party/kissfft
    )

[google-cla[bot]]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[rascani]

Hi, thank you for the PR. TFLM actually does not need array.cc/.h. These files are part of TFLite, and get copied to TFLM for "reasons", but they aren't really needed for compiling our code. Those files use dynamic memory allocation, which we don't allow in TFLM. We specifically exclude those files from the build here.

You'll notice that we actually gate the include based on TF_LITE_STATIC_MEMORY. This #define is basically used within common code for if TFLM. I would suggest adding this define to your CMakeLists.txt, as otherwise things won't built quite right.

I hope that helps!

[kraiskil]

@rascani thanks. That define helped to compile that part. Another compile problem showed up, but I'll continue the discussion in a new issue if I can't figure it out.