This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

patch for Re: arm-rtems vs. arm-elf


Philip Blundell wrote:
> 
> >/tmp/ccEm2a4c.s: Assembler messages:
> >/tmp/ccEm2a4c.s:23: Error: Garbage following instruction
> >/tmp/ccEm2a4c.s:24: Error: Garbage following instruction
> >/tmp/ccEm2a4c.s:25: Error: dest and source1 one must be the same
> >register
> 
> Didn't Richard answer this one a few weeks back?  I think you have to take
> "-Darm" out of your CPP_PREDEFINES so that longlong.h does the right thing.

Yes.  That is the fix. I think Richard did answer this but I don't 
know what I did with the information or patch.  I was chasing the 3.0
release and moving at the same time so probably just messed up.  Sorry.

Here is a patch. It now builds OK.  Thanks.

2001-07-27	Joel Sherrill <joel@OARcorp.com>

	* config/arm/rtems-elf.h: Remove redundant -Darm.

Index: rtems-elf.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/arm/rtems-elf.h,v
retrieving revision 1.1
diff -u -r1.1 rtems-elf.h
--- rtems-elf.h	2000/12/07 02:42:01	1.1
+++ rtems-elf.h	2001/07/27 14:32:44
@@ -26,7 +26,7 @@
 #include "unknown-elf.h"
 
 #undef CPP_PREDEFINES
-#define CPP_PREDEFINES "-Darm -Darm_elf -Drtems -D__rtems__ -D__ELF__ \
+#define CPP_PREDEFINES "-Darm_elf -Drtems -D__rtems__ -D__ELF__ \
    -Asystem(rtems) -Acpu(arm) -Amachine(arm)"
 
 /*#undef INVOKE_main*/


On a more general note, the arm/rtems-elf.h has symptoms
of two other possible cleanups.

==================================
#undef CPP_PREDEFINES
#define CPP_PREDEFINES "-Darm_elf -Drtems -D__rtems__ -D__ELF__ \
   -Asystem(rtems) -Acpu(arm) -Amachine(arm)"

I recall being told that -Drtems and -D__rtems__ are redundant. 
Should one of them be removed.  
===============

The file ends with..

/* Get machine-independent configuration parameters for RTEMS.  */
#include <rtems.h>

Isn't the proper thing to do now is specify multiple tm_files
in configure.in?  
=================

I know these are not critical items but I want to make a
sweep on at least the main trunk and clean things up.

Thanks.

> p.
> 
>   ------------------------------------------------------------------------
>    Part 1.2Type: application/pgp-signature

-- 
Joel Sherrill, Ph.D.             Director of Research & Development
joel@OARcorp.com                 On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
   Support Available             (256) 722-9985


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]