Skip to content

Fix known clang issues in LmP#1006

Merged
MrCry0 merged 3 commits intofoundriesio:mainfrom
MrCry0:main-llvm-fix
Jan 17, 2023
Merged

Fix known clang issues in LmP#1006
MrCry0 merged 3 commits intofoundriesio:mainfrom
MrCry0:main-llvm-fix

Conversation

@MrCry0
Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 commented Jan 17, 2023

It supersedes PR #1003

  • Add strip in clang excludes for Linux kernels
  • Add a description and references to meta-clang
  • Sort a list

quaresmajose and others added 3 commits January 17, 2023 09:07
This fixes errors like:
aarch64-lmp-linux-llvm-strip: error: Link field value 30 in section .rela.dyn is not a symbol table

Ref: meta-clang:kirkstone commit 299a5fd ("linux-yocto: Use binutils provided strip")

Signed-off-by: Oleksandr Suvorov <[email protected]>
@angolini
Copy link
Copy Markdown
Contributor

Do we know the root cause of those errors?

I don't see anything to comment on the commits, but I feel like a more verbose explanation might be helpful in the future.

Copy link
Copy Markdown
Member

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM

@MrCry0 MrCry0 merged commit f698342 into foundriesio:main Jan 17, 2023
@quaresmajose
Copy link
Copy Markdown
Member

Do we know the root cause of those errors?

I don't see anything to comment on the commits, but I feel like a more verbose explanation might be helpful in the future.

This is introduced somewhere in oe-core kirkstone but don't know yet where.

@MrCry0 MrCry0 deleted the main-llvm-fix branch January 24, 2023 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants