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

        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

gcc/ChangeLog

+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.

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)

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"