This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH], PR ipa/81238, make default target_clone static


In going through the target_clone support for the PowerPC, I noticed
that the default target clone function was not explicitly set to private.  This
patch fixes this.  I have checked both the x86_64 and PowerPC target clone
code, and the patch does make it private.

I'm doing bootstrap builds on both little endian power8 and an x86_64 system
right now (both builds are at stage3, so I don't expect any problems).
Assuming there is no regression, can I check this patch in to the trunk?

2017-06-27  Michael Meissner  <meissner@linux.vnet.ibm.com>

	PR ipa/81238
	* multiple_target.c (create_dispatcher_calls): Set the default
	clone to be static, not public.

Index: gcc/multiple_target.c
===================================================================
--- gcc/multiple_target.c	(revision 249710)
+++ gcc/multiple_target.c	(working copy)
@@ -148,6 +148,7 @@ create_dispatcher_calls (struct cgraph_n
 	}
     }
 
+  TREE_PUBLIC (node->decl) = 0;
   symtab->change_decl_assembler_name (node->decl,
 				      clone_function_name (node->decl,
 							   "default"));

-- 
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meissner@linux.vnet.ibm.com, phone: +1 (978) 899-4797


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]