Skip to content
This repository has been archived by the owner on Oct 25, 2023. It is now read-only.

[CI] fix unity build failure related to undefined symbols in tflite #38

Merged
merged 1 commit into from
Mar 2, 2023

Conversation

vvchernov
Copy link
Contributor

@vvchernov vvchernov commented Mar 2, 2023

Currently CI fails during tflite build with the next issue:

[2023-03-02T07:44:01.761Z] /opt/tflite/_deps/cpuinfo-build/libcpuinfo.a(api.c.o): In function cpuinfo_log_fatal.constprop.0': [2023-03-02T07:44:01.761Z] api.c:(.text+0xa3): undefined reference to clog_vlog_fatal'
[2023-03-02T07:44:01.761Z] /opt/tflite/_deps/cpuinfo-build/libcpuinfo.a(init.c.o): In function cpuinfo_log_error': [2023-03-02T07:44:01.761Z] init.c:(.text+0x9f): undefined reference to clog_vlog_error'
[2023-03-02T07:44:01.761Z] /opt/tflite/_deps/cpuinfo-build/libcpuinfo.a(processors.c.o): In function cpuinfo_log_error': [2023-03-02T07:44:01.761Z] processors.c:(.text+0x10f): undefined reference to clog_vlog_error'
[2023-03-02T07:44:01.761Z] /opt/tflite/_deps/cpuinfo-build/libcpuinfo.a(smallfile.c.o): In function cpuinfo_log_error.constprop.0': [2023-03-02T07:44:01.761Z] smallfile.c:(.text+0xa3): undefined reference to clog_vlog_error'
[2023-03-02T07:44:01.761Z] /usr/bin/ld: libtvm_runtime.so: internal symbol `clog_vlog_fatal' isn't defined
[2023-03-02T07:44:01.761Z] /usr/bin/ld: final link failed: Bad value
[2023-03-02T07:44:01.761Z] collect2: error: ld returned 1 exit status

Fix was done based on similar issue for Caffe2

By product: lint fixes were done

@vvchernov vvchernov force-pushed the vc/fix-ci-unity-build branch 3 times, most recently from af7c50c to d3b61d1 Compare March 2, 2023 12:00
@vvchernov
Copy link
Contributor Author

@jwfromm could you see and merged this PR as intermediate fix of CI pipeline?

@vvchernov vvchernov changed the title WIP: [CI] fix unity build [CI] fix unity build failure related to undefined symbols in tflite Mar 2, 2023
Copy link
Contributor

@jwfromm jwfromm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@jwfromm jwfromm merged commit a80dd38 into octoml:relax Mar 2, 2023
@vvchernov vvchernov deleted the vc/fix-ci-unity-build branch March 3, 2023 05:44
vinx13 pushed a commit to vinx13/relax-octo that referenced this pull request Mar 29, 2023
* Update shape lowering pass.

* Rebase.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants