-
Notifications
You must be signed in to change notification settings - Fork 186
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
nonclangable.conf: Add new exceptions #702
Conversation
Add "pciutils" for riscv64 architecture Add "erlang" for all architectures (found on riscv64 and x86-64) Add "grub" for x86-64 architecture Use proper OBJCOPY for "linux-yocto" for all architectures Signed-off-by: Aleksey Smirnov <aleksey.smirnov@yadro.com>
@@ -1,3 +1,20 @@ | |||
#| clang-15: error: unable to execute command: Segmentation fault (core dumped) | |||
#| clang-15: error: linker command failed due to signal (use -v to see invocation) | |||
TOOLCHAIN:pn-pciutils:riscv64 = "gcc" |
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.
I can build pciutils
fine with clang-15 from master branch. So I wonder whats different on your system. I use yoe distribution (https://github.com/yoedistro/yoe-distro)
# run.do_uboot_mkimage.1228: 185: riscv64-yadro-linux-llvm-objcopy: not found | ||
# FileNotFoundError: [Errno 2] No such file or directory: 'x86_64-yadro-linux-llvm-objcopy' | ||
# both riscv64 and x86-64 | ||
OBJCOPY:pn-linux-yocto:toolchain-clang = "${HOST_PREFIX}objcopy" |
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.
There already is similar setting for mips
perhaps we need to remove that also if we set it globally like this.
# both riscv64 and x86-64 | ||
TOOLCHAIN:pn-erlang = "gcc" | ||
|
||
# run.do_uboot_mkimage.1228: 185: riscv64-yadro-linux-llvm-objcopy: not found |
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.
this comment does not match the code, its not uboot but kernel.
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 take a look at the comments
I have reworked this a bit and dropped pciutils change since it works ok with clang15 on master |
Add "pciutils" for riscv64 architecture
Add "erlang" for all architectures (found on riscv64 and x86-64) Add "grub" for x86-64 architecture
Use proper OBJCOPY for "linux-yocto" for all architectures
Signed-off-by: Aleksey Smirnov aleksey.smirnov@yadro.com
Contributor checklist
Signed-off-by
is presentReviewer Guidelines