darwin.c (machopic_symbol_defined_p): In addition to being SYMBOL_REF_LOCAL_P...

	* config/darwin.c (machopic_symbol_defined_p): In addition to
	being SYMBOL_REF_LOCAL_P, a symbol must also be
	! SYMBOL_REF_EXTERNAL_P, as only those are _always_ locally
	defined in this output file, or translation unit in the case
	of IMA not being used.
	(machopic_output_indirection): SYMBOL_REF_LOCAL_P symbols
	never need .indirect_symbol when indirecting.

From-SVN: r86915

gcc/ChangeLog

+2004-09-01  Mike Stump  <mrs@apple.com>
+
+	* config/darwin.c (machopic_symbol_defined_p): In addition to
+	being SYMBOL_REF_LOCAL_P, a symbol must also be
+	! SYMBOL_REF_EXTERNAL_P, as only those are _always_ locally
+	defined in this output file, or translation unit in the case
+	of IMA not being used.
+	(machopic_output_indirection): SYMBOL_REF_LOCAL_P symbols
+	never need .indirect_symbol when indirecting.
+
 2004-09-01  Jakub Jelinek  <jakub@redhat.com>
 
 	* Makefile.in (bb-reorder.o): Add several dependencies.

gcc/config/darwin.c

@@ -97,9 +97,8 @@ name_needs_quotes (const char *name)
 static int
 machopic_symbol_defined_p (rtx sym_ref)
 {
-  return ((SYMBOL_REF_FLAGS (sym_ref) & MACHO_SYMBOL_FLAG_DEFINED)
-	  /* Local symbols must always be defined.  */
-	  || SYMBOL_REF_LOCAL_P (sym_ref));
+  return (SYMBOL_REF_FLAGS (sym_ref) & MACHO_SYMBOL_FLAG_DEFINED)
+    || (SYMBOL_REF_LOCAL_P (sym_ref) && ! SYMBOL_REF_EXTERNAL_P (sym_ref));
 }
 
 /* This module assumes that (const (symbol_ref "foo")) is a legal pic
@@ -874,7 +873,8 @@ machopic_output_indirection (void **slot, void *data)
       machopic_output_stub (asm_out_file, sym, stub);    
     }
   else if (! indirect_data (symbol)
-	   && machopic_symbol_defined_p (symbol))
+	   && (machopic_symbol_defined_p (symbol)
+	       || SYMBOL_REF_LOCAL_P (symbol)))
     {
       data_section ();
       assemble_align (GET_MODE_ALIGNMENT (Pmode));

gcc/testsuite/gcc.dg/visibility-b.c

+/* { dg-do assemble { target *-*-darwin* } } */
+
+#define __private_extern__ extern __attribute__((visibility ("hidden")))
+
+__private_extern__ int n;
+
+int
+mach_error_type(int sub)
+{
+ if (sub >= n)
+     return 1;
+ return 0;
+}