Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] pkg/{flatbuffers,tflite-micro}: upgrade & fix compilation with LLVM #19641

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

maribu
Copy link
Member

@maribu maribu commented May 22, 2023

Contribution description

  • upgrade pkg/flatbuffers and re-enable building with LLVM
  • always use flatc from dist to avoid version mismatch between headers and flatc
  • fix pkg/tflite-micro to build with new pkg/flatbuffers

WARNING:

  • This still includes a commit to disable LLVM, which I expect to land in master soonish. This needs to be rebased on top of master once this is in.
  • Fixing pkg/tflite-micro to build with pkg/flatbuffers is not yet done

Testing procedure

Both packages should build with both GCC and LLVM and the corresponding test apps should pass.

Issues/PRs references

Split out of #19634

@maribu maribu added the State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet label May 22, 2023
@github-actions github-actions bot added Area: pkg Area: External package ports Area: tools Area: Supplementary tools labels May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports Area: tools Area: Supplementary tools State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant