ARM patches applied

Nick Clifton nickc@cygnus.com
Thu Feb 25 10:36:00 GMT 1999


Hi Richard,

: This looks ok now.  I suggest you install it, then we can deal with
: any other matters arising.

I have now done this.  I also applied one other small patch at the
same time to fix a small typo:

Index: gcc/config/arm/arm.h
===================================================================
RCS file: /egcs/carton/cvsfiles/egcs/gcc/config/arm/arm.h,v
retrieving revision 1.24
diff -p -r1.24 arm.h
*** arm.h	1999/02/25 10:20:18	1.24
--- arm.h	1999/02/25 18:34:29
*************** extern int frame_pointer_needed;
*** 98,104 ****
  #if TARGET_CPU_DEFAULT == TARGET_CPU_arm7m
  #define CPP_ARCH_DEFAULT_SPEC "-D__ARM_ARCH_3M__"
  #else
! #if TARGET_CPU_DEFAULT == TARGET_CPU_arm7tdmi || TARGET_CPU_DEFAULT == TARGET_CPU_ARM9
  #define CPP_ARCH_DEFAULT_SPEC "-D__ARM_ARCH_4T__"
  #else
  #if TARGET_CPU_DEFAULT == TARGET_CPU_arm8 || TARGET_CPU_DEFAULT == TARGET_CPU_arm810 || TARGET_CPU_DEFAULT == TARGET_CPU_strongarm
--- 98,104 ----
  #if TARGET_CPU_DEFAULT == TARGET_CPU_arm7m
  #define CPP_ARCH_DEFAULT_SPEC "-D__ARM_ARCH_3M__"
  #else
! #if TARGET_CPU_DEFAULT == TARGET_CPU_arm7tdmi || TARGET_CPU_DEFAULT == TARGET_CPU_arm9
  #define CPP_ARCH_DEFAULT_SPEC "-D__ARM_ARCH_4T__"
  #else
  #if TARGET_CPU_DEFAULT == TARGET_CPU_arm8 || TARGET_CPU_DEFAULT == TARGET_CPU_arm810 || TARGET_CPU_DEFAULT == TARGET_CPU_strongarm



: That might help to tidy up some of the internal code (though it some ways 
: it obfuscates what is happening); but it still misses my point, which is 
: that the user option -mshort-load-bytes is an inaccurate description of 
: what really happens and why we do it.

OK - I will have a look at generating a patch to change the option's
name.

Cheers
	Nick



More information about the Gcc-patches mailing list