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

Support kernel compilation with Clang/LLVM #7086

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

Randl
Copy link
Contributor

@Randl Randl commented Aug 14, 2024

Description

The kernel supports compilation with LLVM in addition to GCC, including cross-compilation to arm and riscv. This PR adds an option to use KERNEL_COMPILER=clang to compile kernel with clang.

Documentation summary for feature / change

I haven't added documentation but probably should?

How Has This Been Tested?

  • Built armbian with clang for one target of each architecture (x86, arm, riscv)
  • Verified that gcc build still works

Checklist:

Please delete options that are not relevant.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

@Randl Randl requested a review from a team as a code owner August 14, 2024 09:39
@github-actions github-actions bot added size/small PR with less then 50 lines Framework Framework components labels Aug 14, 2024
@igorpecovnik igorpecovnik added 11 Milestone: Fourth quarter release Needs Documentation New feature needs documentation entry Giveaway Code completition was selected to enter contributors giveaway labels Aug 14, 2024
@igorpecovnik
Copy link
Member

I haven't added documentation but probably should?

That would be helpful. We need to improve our documentation. Upper part is good while below is chaos https://docs.armbian.com/Developer-Guide_Build-Options/#build-options-below-needs-to-be-retested-and-added-above-could-be-deprecated and I hope someone finds this ticket ;)

@Randl
Copy link
Contributor Author

Randl commented Aug 15, 2024

So if I understand correctly, this should be a separate PR to the https://github.com/armbian/documentation/ repo?

@adeepn
Copy link
Member

adeepn commented Aug 15, 2024

So if I understand correctly, this should be a separate PR to the https://github.com/armbian/documentation/ repo?

Yes

@igorpecovnik igorpecovnik added the Ready to merge Reviewed, tested and ready for merge label Aug 15, 2024
@igorpecovnik igorpecovnik merged commit 0aa2371 into armbian:main Aug 15, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Milestone: Fourth quarter release Framework Framework components Giveaway Code completition was selected to enter contributors giveaway Needs Documentation New feature needs documentation entry Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

3 participants