areusch commented on a change in pull request #6950:
URL: https://github.com/apache/tvm/pull/6950#discussion_r529707962
##########
File path: python/tvm/micro/build.py
##########
@@ -162,7 +158,12 @@ def build_static_runtime(
libs.append(compiler.library(lib_build_dir, lib_srcs, lib_opts))
- libs.append(compiler.library(mod_build_dir, [mod_src_path],
generated_lib_opts))
+ libs.append(
Review comment:
I think in the future we should consider generating full projects (I.e.
zephyr projects) at which point we should have a defined way to describe module
and library sources to the generators. saving generated code to the workspace
tree means that (if the Workspace is operating with debug=True, you get source
included in your debugger. that can come in handy when the thing causing a
CHECK-fail is memory overflow from a TVMBackendAllocWorkspace inside an
operator function. So I think we want A1 but we also want to compile the
library from the user-accessible source files.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
