search

tum-ei-eda / mlonmcu-sw

Target software library (MLIF, Machine Learning Interface) used by the MLonMCU python package
1 stars 3 forks source link

Add llvm support for pulpissimo #7

Closed zhouxinyu0723 closed 1 year ago

zhouxinyu0723 commented 1 year ago

Should I also apply the changes to the target pulp. The current changes are for pulpissimo

PhilippvK commented 1 year ago

Should I also apply the changes to the target pulp. The current changes are for pulpissimo

I would like to test the pulpissimo patch first.

PhilippvK commented 1 year ago

@zhouxinyu0723 I pushed a few updates to you branch. The Pulpissimo support seems to work like expected. Feel free to create the pulp version of the toolchain now as well.

zhouxinyu0723 commented 1 year ago

A question, should the "-fuse-ld=lld" be added to the link flags?

PhilippvK commented 1 year ago

A question, should the "-fuse-ld=lld" be added to the link flags?

@zhouxinyu0723 Yes!

PhilippvK commented 1 year ago

Will you add the pulp patch here or to a new pr?

zhouxinyu0723 commented 1 year ago

Will you add the pulp patch here or to a new pr?

I would like to add it here because they are very similar to each other.

PhilippvK commented 1 year ago

@zhouxinyu0723 ready to merge or is llvm support for PULP missing?

zhouxinyu0723 commented 1 year ago

LLVM support for pulp is also within this PR. Basically, cmake/toolchains/llvm_gvsoc_pulp.cmake is copied from cmake/toolchains/llvm_gvsoc_pulpissimo.cmake. The only difference between pulp and pulpissmio in mlonmcu-sw repo is the BSP packages they are using