Re: recent arm.h change

[Nick Clifton <nickc at cygnus dot com>]

Hi Richard,

: Your recent change on the branch
: 
: 2000-03-24  Nick Clifton  <nickc@cygnus.com>
: 
:         * config/arm/arm.h (SUBTARGET_EXTRA_ASM_SPEC): Define if not
:         already defined.
:         (ASM_SPEC): Define if not already defined.
:         (ASM_OUTPUT_DEF_FROM_DECLS): Define.
: 
: breaks builds on a.out assemblers.  These assemblers don't support all the 
: flags that this defines, even in the latest builds.  The defines need to 
: be moved to the elf (and maybe coff) SUBTARGET_EXTRA_ASM_SPEC defines.  We 
: also need to check that there is at least one official release of GAS that 
: supports these before we can add them at all.
: 
: I don't believe it is acceptable for gcc to require a development version 
: of gas be used.

I agree with that, but I still ike the idea of simplifying the mess of
header arm files by unifying the definitionsof ASM_SPEC.  Wouldn't it
just be simpler to suppress the definition of ASM_SPEC if the host
asssembler is an a.out assembler ?  ie something like:


  #if !defined ASM_SPEC && !defined AOUT
  #define ASM_SPEC "\
  %{mbig-endian:-EB} \
  %{mcpu=*:-m%*} \
  %{march=*:-m%*} \
  %{mapcs-*:-mapcs-%*} \
  %{matpcs:-matpcs} \
  %{mapcs-float:-mfloat} \
  %{msoft-float:-mno-fpu} \
  %{mthumb-interwork:-mthumb-interwork} \
  " SUBTARGET_EXTRA_ASM_SPEC
  #endif

[I do not know which cpp symbol will be defined by a.out hosts, so I
guessed in the example above].

Cheers
	Nick