This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, mips] Fix -mno-shared option (gcc.target/mips/pr35802.c)
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: "Steve Ellcey " <sellcey at mips dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 10 Dec 2012 19:41:54 +0000
- Subject: Re: [patch, mips] Fix -mno-shared option (gcc.target/mips/pr35802.c)
- References: <c3a6726f-5463-4d16-b21b-79235bea6d69@EXCHHUB01.MIPS.com>
"Steve Ellcey " <sellcey@mips.com> writes:
> The test gcc.target/mips/pr35802.c is failing with the message:
>
> xgcc: error: unrecognized command line option '-mdsp-mno-shared'
>
> Obviously the -mdsp and -mno-shared flags got stuck together without
> a space between them. This patch fixes the failure. Ok to checkin?
> It was tested with no regressions.
Ugh.
> diff --git a/gcc/config/mips/gnu-user.h b/gcc/config/mips/gnu-user.h
> index d35ddac..55ca662 100644
> --- a/gcc/config/mips/gnu-user.h
> +++ b/gcc/config/mips/gnu-user.h
> @@ -97,7 +97,7 @@ along with GCC; see the file COPYING3. If not see
> #ifdef HAVE_AS_NO_SHARED
> /* Default to -mno-shared for non-PIC. */
> # define NO_SHARED_SPECS \
> - "%{mshared|mno-shared|fpic|fPIC|fpie|fPIE:;:-mno-shared}"
> + "%{mshared|mno-shared|fpic|fPIC|fpie|fPIE:;: -mno-shared}"
> #else
> # define NO_SHARED_SPECS ""
> #endif
MARCH_MTUNE_NATIVE_SPECS puts the space after the opening quote,
so we might as well do the same here for consistency. OK with that
change, thanks.
Richard