re PR lto/52400 (lto1: ICE with extern on static linkage)

2012-02-28  Richard Guenther  <rguenther@suse.de>

	PR lto/52400
	* lto.c (lto_register_function_decl_in_symtab): Do not register
	a reverse renamed decl mapping.

	* g++.dg/lto/pr52400_0.C: New testcase.

From-SVN: r184618

gcc/lto/ChangeLog

+2012-02-28  Richard Guenther  <rguenther@suse.de>
+
+	PR lto/52400
+	* lto.c (lto_register_function_decl_in_symtab): Do not register
+	a reverse renamed decl mapping.
+
 2012-01-06  Jakub Jelinek  <jakub@redhat.com>
 
 	PR lto/51774

gcc/lto/lto.c

@@ -689,13 +689,6 @@ lto_register_function_decl_in_symtab (struct data_in *data_in, tree decl)
 	  lto_record_renamed_decl (data_in->file_data,
 				   IDENTIFIER_POINTER (old_assembler_name),
 				   IDENTIFIER_POINTER (new_assembler_name));
-
-	  /* Also register the reverse mapping so that we can find the
-	     new name given to an existing assembler name (used when
-	     restoring alias pairs in input_constructors_or_inits.  */
-	  lto_record_renamed_decl (data_in->file_data,
-				   IDENTIFIER_POINTER (new_assembler_name),
-				   IDENTIFIER_POINTER (old_assembler_name));
 	}
     }
 

gcc/testsuite/ChangeLog

+2012-02-28  Richard Guenther  <rguenther@suse.de>
+
+	PR lto/52400
+	* g++.dg/lto/pr52400_0.C: New testcase.
+
 2012-02-28  Georg-Johann Lay  <avr@gjlay.de>
 
 	* gcc.target/avr/torture/builtins-1.c: New test.

gcc/testsuite/g++.dg/lto/pr52400_0.C

+// { dg-lto-do run }
+
+extern "C" {
+  static int f4(int);
+
+int f5(int a) {
+  extern int f4(int);
+  return f4(a);
+}
+}
+
+int f4(int a) { return 4+a; }
+
+int main(int argc, char *argv[])
+{
+  int a = f4(1);
+  return !(a == 5);
+}