On 13 Oct 2011, at 23:38, Iain Sandoe wrote:
On 13 Oct 2011, at 23:22, Mike Stump wrote:
+/* Add $LDBL128 suffix to long double builtins for ppc darwin. */
static void
-darwin_patch_builtin (int fncode)
+darwin_patch_builtin (enum built_in_function fncode)
This is a property of the target machine. DARWIN_PPC is a property
of the target machine; maybe if (DARWIN_PPC) { } will do what you
want?
yes, that should be right - there was no reason that this should
have broken x86 darwin.
regstrapped on *-darwin9, x86-64-darwin10, applied as r179962.
gcc:
PR bootstrap/50699
* config/darwin.c (darwin_patch_builtin): Adjust argument type. Only
build for powerpc targets. (darwin_patch_builtins): Only build for
powerpc targets.
Index: gcc/config/darwin.c
===================================================================
--- gcc/config/darwin.c (revision 179961)
+++ gcc/config/darwin.c (working copy)
@@ -2957,10 +2957,11 @@ darwin_override_options (void)
darwin_running_cxx = (strstr (lang_hooks.name, "C++") != 0);
}
-/* Add $LDBL128 suffix to long double builtins. */
+#if DARWIN_PPC
+/* Add $LDBL128 suffix to long double builtins for ppc darwin. */
static void
-darwin_patch_builtin (int fncode)
+darwin_patch_builtin (enum built_in_function fncode)
{
tree fn = builtin_decl_explicit (fncode);
tree sym;
@@ -2998,6 +2999,7 @@ darwin_patch_builtins (void)
#undef PATCH_BUILTIN_NO64
#undef PATCH_BUILTIN_VARIADIC
}
+#endif
/* CFStrings implementation. */
static GTY(()) tree cfstring_class_reference = NULL_TREE;
