[PATCH 2/2] RISC-V: Add riscv{32, 64}be with big endian as default
Kito Cheng
kito.cheng@gmail.com
Wed Feb 3 06:12:52 GMT 2021
> > Could you extract the endian related LINK_SPEC change to
> > ENDIAN_LINK_SPEC to riscv.h, so that we can prevent
> > duplicate this several times.
>
> You mean a define which expands to
>
> "-melf" XLEN_SPEC DEFAULT_ENDIAN_SPEC "riscv"
Yeah, but I'd like to include following 2 lines too:
%{mbig-endian:-EB} \
%{mlittle-endian:-EL} \
I saw it's just the same among 3 files.
>
> ? Sure, but I don't think ENDIAN_LINK_SPEC would be a good name for
> it since it defines the word size as well as the endianness, and also
> ELF in general.
>
> Maybe ELF_LINK_SPEC? The word size and endianness are also ELF
> properties (as encoded in EI_CLASS and EI_DATA).
Either ENDIAN_LINK_SPEC or ELF_LINK_SPEC is ok to me,
I don't have strong preference on nanming.
>
>
> // Marcus
>
>
More information about the Gcc-patches
mailing list