On 27/07/17 21:57, Segher Boessenkool wrote:

On Thu, Jul 27, 2017 at 07:28:30AM +0200, Sebastian Huber wrote:
This deletes eabi.h and I don't see you add all its definitions to
rtems.h directly (NAME__MAIN etc.)  Is this on purpose?
Yes, I always wondered why GCC added the __eabi() call to main() out of
thin air. In general, there is no main() function in RTEMS. Instead, you
can statically configure initialization threads. We call __eabi() in the
low-level startup code, e.g.

https://git.rtems.org/rtems/tree/c/src/lib/libbsp/powerpc/qoriq/start/start.S#n144
Heh, I always thought the EABI must require it, but it seems to be a GCC
invention.

Patch looks fine to me then.  You can approve it yourself of course:-)

Thanks for your review, I checked it in in mainline and 7 branch.

Would you mind having a look at:

https://gcc.gnu.org/ml/gcc/2017-07/msg00196.html

Is the small data area no longer supported by GCC for 64-bit PowerPC?

--
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.hu...@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.

Reply via email to