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

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

Open
Nelson1225 opened this issue Dec 22, 2020 · 5 comments

Comments

@Nelson1225
Copy link

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

@nick-knight
Copy link

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.")

@Nelson1225
Copy link
Author

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.

@Nelson1225 Nelson1225 changed the title New assembler options, -mbig-endian and -mlittle-endian, used to choose the endian. New compiler options, -mbig-endian and -mlittle-endian, used to choose the endian. Dec 22, 2020
@Nelson1225
Copy link
Author

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

@Nelson1225 Nelson1225 reopened this Dec 22, 2020
@jim-wilson
Copy link
Collaborator

I am in favor of accepting these new compiler options.

@ebahapo
Copy link

ebahapo commented Jan 19, 2021

I second this proposal.

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

No branches or pull requests

4 participants