Richard Sandiford <rdsandif...@googlemail.com> writes:
> Robert Suchanek <robert.sucha...@imgtec.com> writes:
> > @@ -771,7 +771,8 @@ struct mips_cpu_info {
> >
> > /* Infer a -mnan=2008 setting from a -mips argument. */
> > #define MIPS_ISA_NAN2008_SPEC \
> > - "%{mnan*:;mips32r6|mips64r6:-mnan=2008}"
> > + "%{mnan*:;mips32r6|mips64r6:-mnan=2008;march=m51*: \
> > + %{!msoft-float:-mnan=2008}}"
>
> Did you need this, or was it for completeness? MIPS_ISA_NAN2008_SPEC
> should only be used after MIPS_ISA_LEVEL_SPEC, so I would have expected
> the mips32r6|mips64r6: case to fire for -march=m51* too, ahead of the
> new case.
The m5100 is a MIPS32R5 but with a NAN2008 FPU which is why there is the
special case. The soft-float case is to try and limit the number of
multilib variants required so we stick to nan legacy for softfloat by
default.
Thanks,
Matthew
