riscv-non-isa/riscv-c-api-doc

New compiler options, -mbig-endian and -mlittle-endian, used to choose the endian.

Nelson1225 opened this issue · 5 comments

Hi Guys,

We get the GNU binutils patches from Marcus Comstedt recently, to support RISCV big endian targets [1]. In the patches, we need to add new assembler options, -mbig-endian and -mlittle-endian, to choose the endian in assembler. I have raised another issue to discuss the new target triplet in the psabi, riscv-non-isa/riscv-elf-psabi-doc#167. Once we all agree with these changes, we will need to add the new assembler options, -mbig-endian and -mlittle-endian, into the c-api-doc, too.

Thanks
Nelson

[1] https://sourceware.org/pipermail/binutils/2020-December/114632.html

Hi @Nelson1225 , thanks for following up on this.

Are assembler command-line options really part of the "C API"? I think this might belong over at https://github.com/riscv/riscv-toolchain-conventions ("This effort aims to document the expected behaviour and command-line interface of RISC-V toolchains.")

Oh thanks for pointing this out, @knightsifive. I see the description, and we should add the options to the binutils doc directly, if the patches are accepted. I will open a new issue to riscv-toolchain-conventions and close this one.

Reopen it since we also need the compiler options. Kito had reminded me before that the following link,
riscv-software-src/riscv-isa-sim#575

I am in favor of accepting these new compiler options.

I second this proposal.