Remove special handling of method named during preloading

Nowadays op_arrays always hold a ref to the function_name, even
if they are fully shared. As such, I don't believe that this code
is necessary anymore, and it interferes with the code that
releases function_names the appropriate number of times while
persisting.

Author: nikic

ext/opcache/ZendAccelerator.c

@@ -4707,8 +4707,6 @@ static int accel_preload(const char *config, bool in_child)
 		zend_persistent_script *script;
 		int ping_auto_globals_mask;
 		int i;
-		zend_class_entry *ce;
-		zend_op_array *op_array;
 
 		if (PG(auto_globals_jit)) {
 			ping_auto_globals_mask = zend_accel_get_auto_globals();
@@ -4878,19 +4876,6 @@ static int accel_preload(const char *config, bool in_child)
 		HANDLE_BLOCK_INTERRUPTIONS();
 		SHM_UNPROTECT();
 
-		/* Store method names first, because they may be shared between preloaded and non-preloaded classes */
-		ZEND_HASH_FOREACH_PTR(&script->script.class_table, ce) {
-			ZEND_HASH_FOREACH_PTR(&ce->function_table, op_array) {
-				zend_string *new_name = zend_shared_alloc_get_xlat_entry(op_array->function_name);
-
-				if (!new_name) {
-					new_name = accel_new_interned_string(op_array->function_name);
-					zend_shared_alloc_register_xlat_entry(op_array->function_name, new_name);
-				}
-				op_array->function_name = new_name;
-			} ZEND_HASH_FOREACH_END();
-		} ZEND_HASH_FOREACH_END();
-
 		ZCSG(preload_script) = preload_script_in_shared_memory(script);
 
 		SHM_PROTECT();