This is the mail archive of the
mailing list for the GCC project.
RE: [Patch] SH : Add target specific option -msort-data for SH-ELFtarget
- From: "Anil Paranjape" <AnilP1 at KPITCummins dot com>
- To: "Zack Weinberg" <zack at codesourcery dot com>
- Cc: "Richard Sandiford" <richard at codesourcery dot com>,<gcc-patches at gcc dot gnu dot org>
- Date: Thu, 23 Jun 2005 15:03:44 +0530
- Subject: RE: [Patch] SH : Add target specific option -msort-data for SH-ELFtarget
Thank you for your comments.
It would be really useful if this is done target independent.
As far as my knowledge is concerned, it requires change in
How to relate this feature with "-Os" option so as not to enable it by
Kindly guide me on how to write this patch so that it will be target
From: Zack Weinberg [mailto:firstname.lastname@example.org]
Sent: Wednesday, June 22, 2005 7:40 PM
To: Anil Paranjape
Cc: Richard Sandiford; email@example.com
Subject: Re: [Patch] SH : Add target specific option -msort-data for
Anil Paranjape wrote:
> Hi Richard,
> Thank you for your valuable suggestions.
> We have implemented your suggestions and modified the patch
> Now, as you have said, no modification in linker script is required.
> Any existing linker script that use "*(.data*)" and "*(.rodata*)" will
Why is this coded as a SH-specific patch? It would be useful on any
I could argue for enabling it as part of -Os, even. (One wouldn't want
to do it by default, as this layout might not be maximally
> +#if __ELF__
> + default_elf_select_section (decl, reloc, align); #else
> + default_select_section (decl, reloc, align); #endif
Never do this. __ELF__ is a property of the host, not the target.