Here is the patch that uses the arm-fpus.def list. Thanks Yvan
2014-08-27 Yvan Roux <yvan.r...@linaro.org> * config.gcc (arm*-*-*): Check --with-fpu against arm-fpus.def. On 27 August 2014 12:35, Yvan Roux <yvan.r...@linaro.org> wrote: > On 27 August 2014 11:24, Richard Earnshaw <rearn...@arm.com> wrote: >> On 27/08/14 09:04, Yvan Roux wrote: >>> Hi, >>> >>> as reported in PR62248 there is a typo in gcc/config.gcc where >>> --with-fpu doesn't match -mfpu option for fp-armv8 value (fp-arm-v8 in >>> config.gcc). Here is the patch to fix it. >>> >>> Thanks, >>> Yvan >>> >>> 2014-08-27 Yvan Roux <yvan.r...@linaro.org> >>> >>> * config.gcc: Fix fp-armv8 option for arm*-*-* targets. >>> >>> >>> pr62248.diff >>> >>> >>> diff --git a/gcc/config.gcc b/gcc/config.gcc >>> index 6862c127..3f68e3e 100644 >>> --- a/gcc/config.gcc >>> +++ b/gcc/config.gcc >>> @@ -3528,7 +3528,7 @@ case "${target}" in >>> | vfp | vfp3 | vfpv3 \ >>> | vfpv3-fp16 | vfpv3-d16 | vfpv3-d16-fp16 | vfpv3xd \ >>> | vfpv3xd-fp16 | neon | neon-fp16 | vfpv4 | vfpv4-d16 \ >>> - | fpv4-sp-d16 | neon-vfpv4 | fp-arm-v8 | neon-fp-armv8 \ >>> + | fpv4-sp-d16 | neon-vfpv4 | fp-armv8 | neon-fp-armv8 \ >>> | crypto-neon-fp-armv8) >>> # OK >>> ;; >>> >> >> Ok; but better still would be to change this to use the official list in >> arm-fpus.def (like we do for CPU names). > > Yes indeed, I'll do it that way.
diff --git a/gcc/config.gcc b/gcc/config.gcc index 6862c127..7434a08 100644 --- a/gcc/config.gcc +++ b/gcc/config.gcc @@ -3523,20 +3523,17 @@ case "${target}" in ;; esac - case "$with_fpu" in - "" \ - | vfp | vfp3 | vfpv3 \ - | vfpv3-fp16 | vfpv3-d16 | vfpv3-d16-fp16 | vfpv3xd \ - | vfpv3xd-fp16 | neon | neon-fp16 | vfpv4 | vfpv4-d16 \ - | fpv4-sp-d16 | neon-vfpv4 | fp-arm-v8 | neon-fp-armv8 \ - | crypto-neon-fp-armv8) - # OK - ;; - *) - echo "Unknown fpu used in --with-fpu=$with_fpu" 2>&1 - exit 1 - ;; - esac + # see if it matches any of the entries in arm-fpus.def + if [ x"$with_fpu" = x ] \ + || grep "^ARM_FPU(\"$with_fpu\"," \ + ${srcdir}/config/arm/arm-fpus.def \ + > /dev/null; then + # OK + true + else + echo "Unknown fpu used in --with-fpu=$with_fpu" 1>&2 + exit 1 + fi case "$with_abi" in "" \