> > 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 > >