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

build: Add riscv32 and riscv64 support #194

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

zetalog
Copy link
Contributor

@zetalog zetalog commented Jun 22, 2022

This patch adds riscv32/riscv64 cross compilation support. Notice
that this patch only enforces default psABI for RV32G and RV64G.

Reference: https://github.com/riscv-non-isa/riscv-elf-psabi-doc/blob/master/riscv-cc.adoc
Signed-off-by: Lv Zheng zhenglv@smart-core.cn

This patch adds riscv32/riscv64 cross compilation support. Notice
that this patch only enforces default psABI for RV32G and RV64G.

Reference: https://github.com/riscv-non-isa/riscv-elf-psabi-doc/blob/master/riscv-cc.adoc
Signed-off-by: Lv Zheng <zhenglv@smart-core.cn>
@sbahra sbahra merged commit 03b42ea into concurrencykit:master Jul 11, 2022
@jrtc27
Copy link

jrtc27 commented Jul 11, 2022

Why is this using ilp32 and lp64 when lp64d is the standard ABI to use for Unix things (lp64 is for when you don't have an FPU) and the default ABI for FreeBSD and 64-bit Linux, and ilp32d for 32-bit Linux (strictly, anything other than bare-metal -unknown-elf triples default to double-precision hard float ABIs)?

@jrtc27
Copy link

jrtc27 commented Jul 11, 2022

In fact, why even hard-code an ABI? You should really just be picking up the toolchain's default.

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.

3 participants