-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Arm32 v8m support cmake clang compile #16627
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use
arch/ in the commit title
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for c6dbc8a
please use as title
tools/ci: Compile mps3-an547/gcov with clang using cmake in ci
ok |
The modification to compile ci cmake clang does not seem to have taken effect. How can I make the necessary changes? |
546c886
to
2a2a3a8
Compare
@W-M-R please fix the build break. |
OK |
1. -target should be in front, otherwise clang will not be able to find the corresponding libgcc.a 2. When using clang++ compiler to link, you also need to specify the corresponding arch, otherwise ld.lld will report an error due to arch mismatch. Signed-off-by: wangmingrong1 <wangmingrong1@xiaomi.com>
The target is wrong Signed-off-by: wangmingrong1 <wangmingrong1@xiaomi.com>
armv8m ci: 1. use makefile clang: /arm/mps/mps3-an547/configs/clang,CONFIG_ARM_TOOLCHAIN_CLANG 2. use cmake clang: CMAKE,mps3-an547:gcov Signed-off-by: wangmingrong1 <wangmingrong1@xiaomi.com>
Signed-off-by: wangmingrong1 <wangmingrong1@xiaomi.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add commit messages
Note: Please adhere to Contributing Guidelines.
Summary
Fix some errors and add cmake clang compilation check for armv8m in ci
Impact
None
Testing
ci