RTEMS: Prefer int for int32_t
authorSebastian Huber <sebastian.huber@embedded-brains.de>
Thu, 14 Jun 2018 05:10:51 +0000 (05:10 +0000)
committerSebastian Huber <sh@gcc.gnu.org>
Thu, 14 Jun 2018 05:10:51 +0000 (05:10 +0000)
Common systems like glibc and FreeBSD define int32_t to int.  This means
a lot of third party code works well in these cases:

  #include <stdint.h>

  void f(int32_t);

  void f(int);

  void g(int32_t *);

  void h(void)
  {
    int i;
    g(&i);
  }

On RTEMS you got however in C

  test.c:5:6: error: conflicting types for 'f'
    void f(int);
        ^
  test.c:3:6: note: previous declaration of 'f' was here
    void f(int32_t);
        ^
  test.c: In function 'h':
  test.c:12:4: warning: passing argument 1 of 'g' from incompatible
  pointer type [-Wincompatible-pointer-types]
    g(&i);
      ^
  test.c:7:6: note: expected 'int32_t * {aka long int *}' but argument
  is of type 'int *' void g(int32_t *);

and C++

  test.c: In function 'void h()':
  test.c:12:4: error: invalid conversion from 'int*' to 'int32_t* {aka
  long int*}' [-fpermissive]
    g(&i);
      ^~
  test.c:7:6: note:   initializing argument 1 of 'void g(int32_t*)'
    void g(int32_t *);
      ^

This was due to a Newlib speciality which uses long for int32_t if long
is a 32-bit type.  To ease the use of third party software in RTEMS we
override this Newlib option now and use int for int32_t if int is a
32-bit type.

gcc/
* config/rtems.h (STDINT_LONG32): Define.

From-SVN: r261582

gcc/ChangeLog
gcc/config/rtems.h

index 8c655bd75a96302608709cf8a1ce6e73958cee3a..ceb0d665f6897b2e1697f47dab2f830ebcf77abd 100644 (file)
@@ -1,3 +1,7 @@
+2018-06-14  Sebastian Huber  <sebastian.huber@embedded-brains.de>
+
+       * config/rtems.h (STDINT_LONG32): Define.
+
 2018-06-13  Matthew Fortune  <matthew.fortune@mips.com>
            Prachi Godbole  <prachi.godbole@imgtec.com>
 
index dcea95cddade8c5f52a7b3a4344ace0bc6837a5a..af58eb16115906381f2651cf8fe1e82db3e85624 100644 (file)
@@ -48,3 +48,7 @@
  -latomic -lc -lgcc --end-group %{!qnolinkcmds: -T linkcmds%s}}}"
 
 #define TARGET_POSIX_IO
+
+/* Prefer int for int32_t (see stdint-newlib.h).  */
+#undef STDINT_LONG32
+#define STDINT_LONG32 (INT_TYPE_SIZE != 32 && LONG_TYPE_SIZE == 32)