This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [PATCH mips] Pass -msoft-float/-mhard-float flags to GAS
- From: "Moore, Catherine" <Catherine_Moore at mentor dot com>
- To: Steve Ellcey <sellcey at mips dot com>, "matthew dot fortune at imgtec dot com" <matthew dot fortune at imgtec dot com>, "echristo at gmail dot com" <echristo at gmail dot com>, "GCC Patches" <gcc-patches at gcc dot gnu dot org>
- Cc: "Moore, Catherine" <Catherine_Moore at mentor dot com>
- Date: Sat, 9 Aug 2014 15:03:52 +0000
- Subject: RE: [PATCH mips] Pass -msoft-float/-mhard-float flags to GAS
- Authentication-results: sourceware.org; auth=none
- References: <dbb904a0-4070-407c-b3e7-c04703769bb8 at BAMAIL02 dot ba dot imgtec dot org> <1407526900 dot 2601 dot 48 dot camel at ubuntu-sellcey>
> -----Original Message-----
> From: Steve Ellcey [mailto:sellcey@mips.com]
> Sent: Friday, August 08, 2014 3:42 PM
> To: Moore, Catherine; matthew.fortune@imgtec.com; echristo@gmail.com;
>
> 2014-08-08 Steve Ellcey <sellcey@mips.com>
>
> * config/mips/mips.h (ASM_SPEC): Pass float options to assembler.
>
> diff --git a/gcc/config/mips/mips.h b/gcc/config/mips/mips.h index
> 8d7a09f..9a15287 100644
> --- a/gcc/config/mips/mips.h
> +++ b/gcc/config/mips/mips.h
> @@ -1187,6 +1187,8 @@ struct mips_cpu_info { %{mshared} %{mno-
> shared} \ %{msym32} %{mno-sym32} \ %{mtune=*} \
> +%{mhard-float} %{msoft-float} \
> +%{msingle-float} %{mdouble-float} \
> %(subtarget_asm_spec)"
>
> /* Extra switches sometimes passed to the linker. */
>
Hi Steve,
The patch itself looks okay, but perhaps a question for Matthew.
Does the fact that the assembler requires -msoft-float even if .set softfloat is present in the .s file deliberate behavior?
I don't have a problem with passing along the *float* options to gas, but would hope that the .set options were honored as well.
My short test indicated that the .set *float* options were being ignored if the correct command line option wasn't present.
Thanks,
Catherine