This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Table-based default_options_optimization
- From: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
- To: joseph at codesourcery dot com
- Cc: gcc-patches at gcc dot gnu dot org, aoliva at redhat dot com
- Date: Fri, 22 Oct 2010 20:06:25 +0900 (JST)
- Subject: Re: Table-based default_options_optimization
- References: <Pine.LNX.4.64.1010211708380.2491@digraph.polyomino.org.uk>
"Joseph S. Myers" <joseph@codesourcery.com> wrote:
> The SH changes are more complicated than those for other targets.
> Optimization options could enable MASK_SMALLCODE and
> MASK_SAVE_ALL_TARGET_REGS. The former corresponded to a deprecated
> -mspace option; rather than having -Os imply a deprecated option, I
> made the deprecated option into an alias for -Os and changed all the
> TARGET_SMALLCODE tests to test optimize_size. (Some should probably
> move to being on a per-basic-block basis; I leave that to the SH
> maintainers.) There is no option corresponding to
> MASK_SAVE_ALL_TARGET_REGS, and the tables can only represent implying
> options, not settings without a corresponding option, so I moved that
> implication into sh_option_override.
>
> Bootstrapped with no regressions on x86_64-unknown-linux-gnu. Also
> tested building cc1 for crosses to: alpha-linux-gnu arm-eabi avr-elf
> cris-elf crx-elf fr30-elf frv-elf h8300-elf i386-solaris2.10 ia64-elf
> ia64-hp-vms iq2000-elf lm32-elf m32r-elf mcore-elf mep-elf
> microblaze-elf mips-elf mmix-knuth-mmixware mn10300-elf hppa-linux-gnu
> pdp11-none picochip-none powerpc-eabi rx-elf s390-linux-gnu score-elf
> sh-elf sparc-elf xstormy16-elf v850-elf xtensa-elf. (The mn10300
> build was broken for other reasons at the revision I tested:
> "mn10300.md:1798: unknown mode `CC_FLOAT'".) OK to commit?
The SH portion is Ok. Thanks!
Regards,
kaz