re PR target/25477 (builtin functions should use $LDBL128 suffix on darwin when appro...

        PR target/25477
        * config/darwin-ppc-ldouble-patch.def (BUILT_IN_NANL): Add.
        (BUILT_IN_NEXTTOWARD): Remove.
        (BUILT_IN_NEXTTOWARDF): Ditto.
        * config/darwin.c (darwin_patch_builtin): Use ACONCAT instead of
        alloca/strcpy/strcat.  Remove commented-out code.  Fix whitespace.

From-SVN: r132723

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 4eb013e2f4bcae65c241d81a871e46ea60701d75..b58929382d966bc5f80fcfb9b6d5a0376258bad8 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,12 @@
+2008-02-27  Uros Bizjak  <ubizjak@gmail.com>
+
+       PR target/25477
+       * config/darwin-ppc-ldouble-patch.def (BUILT_IN_NANL): Add.
+       (BUILT_IN_NEXTTOWARD): Remove.
+       (BUILT_IN_NEXTTOWARDF): Ditto.
+       * config/darwin.c (darwin_patch_builtin): Use ACONCAT instead of
+       alloca/strcpy/strcat.  Remove commented-out code.  Fix whitespace.
+
 2008-02-27  Tom Tromey  <tromey@redhat.com>
 
        * tree-dump.c (dequeue_and_dump) <FUNCTION_DECL>: Check
@@ -463,8 +472,7 @@
 
 2008-02-25  Jan Beulich  <jbeulich@novell.com>
 
-       * Makefile.in: Also prefix uses of crt0.o and mcrt0.o with
-       $(T).
+       * Makefile.in: Also prefix uses of crt0.o and mcrt0.o with $(T).
        * config/i386/netware-libgcc.exp: Add __bswap?i2,
        __emultls_get_address, __emultls_register_common,
        __floatundi?f, and _Unwind_GetIPInfo.

diff --git a/gcc/config/darwin-ppc-ldouble-patch.def b/gcc/config/darwin-ppc-ldouble-patch.def
index 0d09221e6a393f7ade888fce7e5f1afdd9b1be5f..b532328408a223e4ab1f81634c5887f5b4bc4069 100644 (file)
--- a/gcc/config/darwin-ppc-ldouble-patch.def
+++ b/gcc/config/darwin-ppc-ldouble-patch.def
@@ -59,10 +59,9 @@ PATCH_BUILTIN (BUILT_IN_LOGL)
 PATCH_BUILTIN (BUILT_IN_LRINTL)
 PATCH_BUILTIN (BUILT_IN_LROUNDL)
 PATCH_BUILTIN (BUILT_IN_MODFL)
+PATCH_BUILTIN (BUILT_IN_NANL)
 PATCH_BUILTIN (BUILT_IN_NEARBYINTL)
 PATCH_BUILTIN (BUILT_IN_NEXTAFTERL)
-PATCH_BUILTIN (BUILT_IN_NEXTTOWARD)
-PATCH_BUILTIN (BUILT_IN_NEXTTOWARDF)
 PATCH_BUILTIN (BUILT_IN_NEXTTOWARDL)
 PATCH_BUILTIN (BUILT_IN_POWL)
 PATCH_BUILTIN (BUILT_IN_REMAINDERL)
@@ -78,6 +77,7 @@ PATCH_BUILTIN (BUILT_IN_TANHL)
 PATCH_BUILTIN (BUILT_IN_TANL)
 PATCH_BUILTIN (BUILT_IN_TGAMMAL)
 PATCH_BUILTIN (BUILT_IN_TRUNCL)
+
 PATCH_BUILTIN_NO64 (BUILT_IN_VFPRINTF)
 PATCH_BUILTIN_NO64 (BUILT_IN_VFSCANF)
 PATCH_BUILTIN_NO64 (BUILT_IN_VPRINTF)

diff --git a/gcc/config/darwin.c b/gcc/config/darwin.c
index 13aa021f4c676fca487d447737fc21380a9d1a3e..6c2de252fe3de62307d70f1f43f1f071d6555e21 100644 (file)
--- a/gcc/config/darwin.c
+++ b/gcc/config/darwin.c
@@ -1748,18 +1748,13 @@ darwin_patch_builtin (int fncode)
     return;
 
   sym = DECL_ASSEMBLER_NAME (fn);
-  newname = alloca (IDENTIFIER_LENGTH (sym) + 10);
-  strcpy (newname, "_");
-  strcat (newname, IDENTIFIER_POINTER (sym));
-  strcat (newname, "$LDBL128");
+  newname = ACONCAT (("_", IDENTIFIER_POINTER (sym), "$LDBL128", NULL));
+
   set_user_assembler_name (fn, newname);
-  /*sym = get_identifier (newname);
-  SET_DECL_ASSEMBLER_NAME (fn, sym);*/
 
   fn = implicit_built_in_decls[fncode];
   if (fn)
     set_user_assembler_name (fn, newname);
-    /*SET_DECL_ASSEMBLER_NAME (fn, sym);*/
 }
 
 void
@@ -1769,11 +1764,11 @@ darwin_patch_builtins (void)
     return;
 
 #define PATCH_BUILTIN(fncode) darwin_patch_builtin (fncode);
-#define PATCH_BUILTIN_NO64(fncode) \
-  if (!TARGET_64BIT) \
+#define PATCH_BUILTIN_NO64(fncode)             \
+  if (!TARGET_64BIT)                           \
     darwin_patch_builtin (fncode);
-#define PATCH_BUILTIN_VARIADIC(fncode) \
-  if (!TARGET_64BIT \
+#define PATCH_BUILTIN_VARIADIC(fncode)                           \
+  if (!TARGET_64BIT                                              \
       && (strverscmp (darwin_macosx_version_min, "10.3.9") >= 0)) \
     darwin_patch_builtin (fncode);
 #include "darwin-ppc-ldouble-patch.def"