Patchwork [12/25] Specs cleanup: microblaze

login
register
mail settings
Submitter Joseph S. Myers
Date Jan. 14, 2011, 12:10 a.m.
Message ID <Pine.LNX.4.64.1101140009580.6059@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/78839/
State New
Headers show

Comments

Joseph S. Myers - Jan. 14, 2011, 12:10 a.m.
config/microblaze/microblaze.h has %{microblaze1} in ASM_SPEC - that
is, a -microblaze1 option would be passed to the assembler.  No such
option is in microblaze.opt, so it would cause an error if passed when
compiling a .c file, and in fact the gas port to microblaze doesn't
have such an option either, so the spec is completely useless and this
patch removes it.  OK to commit?

2011-01-13  Joseph Myers  <joseph@codesourcery.com>

	* config/microblaze/microblaze.h (ASM_SPEC): Remove
	%{microblaze1}.
Michael Eager - Jan. 14, 2011, 7:15 a.m.
Joseph S. Myers wrote:
> config/microblaze/microblaze.h has %{microblaze1} in ASM_SPEC - that
> is, a -microblaze1 option would be passed to the assembler.  No such
> option is in microblaze.opt, so it would cause an error if passed when
> compiling a .c file, and in fact the gas port to microblaze doesn't
> have such an option either, so the spec is completely useless and this
> patch removes it.  OK to commit?

OK to commit.

Patch

diff -rupN --exclude=.svn gcc-mainline-11/gcc/config/microblaze/microblaze.h gcc-mainline/gcc/config/microblaze/microblaze.h
--- gcc-mainline-11/gcc/config/microblaze/microblaze.h	2011-01-12 14:13:00.000000000 -0800
+++ gcc-mainline/gcc/config/microblaze/microblaze.h	2011-01-12 14:26:53.000000000 -0800
@@ -77,7 +77,6 @@  extern enum pipeline_type microblaze_pip
 #define TARGET_ASM_SPEC ""
 
 #define ASM_SPEC "\
-%{microblaze1} \
 %(target_asm_spec)"
 
 /* Extra switches sometimes passed to the linker.  */