cheribuild icon indicating copy to clipboard operation
cheribuild copied to clipboard

seL4: Add new compilation targets

Open heshamelmatary opened this issue 1 year ago • 4 comments

heshamelmatary avatar Jul 12 '23 11:07 heshamelmatary

CI failure looks unrelated. Change looks good to me now, although you aren't actually using it yet.

arichardson avatar Jul 24 '23 14:07 arichardson

CI failure looks unrelated. Change looks good to me now, although you aren't actually using it yet.

You can actually build compiler-rt-builtins-sel4-morello-aarch64 and compiler-rt-builtins-sel4-riscv64 with this PR. Also if you add -d it will install "-none--" tools symlinks required for building seL4. But there's still no seL4 target build yet, that I may add in the future.

heshamelmatary avatar Jul 24 '23 14:07 heshamelmatary

CI failure looks unrelated. Change looks good to me now, although you aren't actually using it yet.

You can actually build compiler-rt-builtins-sel4-morello-aarch64 and compiler-rt-builtins-sel4-riscv64 with this PR. Also if you add -d it will install "-none--" tools symlinks required for building seL4. But there's still no seL4 target build yet, that I may add in the future.

True, but couldn't you use the generic baremetal compiler-rt target already? Adding the symlinks should also be possible without the new target info

arichardson avatar Jul 24 '23 14:07 arichardson

CI failure looks unrelated. Change looks good to me now, although you aren't actually using it yet.

You can actually build compiler-rt-builtins-sel4-morello-aarch64 and compiler-rt-builtins-sel4-riscv64 with this PR. Also if you add -d it will install "-none--" tools symlinks required for building seL4. But there's still no seL4 target build yet, that I may add in the future.

True, but couldn't you use the generic baremetal compiler-rt target already? Adding the symlinks should also be possible without the new target info

Yeah I could just use baremetal. But some variants may need different flags (e.g., hard-float vs soft-float which is the default for bare metal?) or extensions as well. I am happy to wait for this PR until I decide if it's a good idea to build seL4 projects for cheribuild or not.

heshamelmatary avatar Jul 26 '23 16:07 heshamelmatary