From e73d965c22db66e6fc825d54b1dc78d20dcfe176 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Mon, 6 Dec 2010 12:44:51 +0000 Subject: [PATCH] Call xatexit with plugin_call_cleanup. 2010-12-06 H.J. Lu Dmitry Gorbachev PR ld/12289 * ldmain.c (main): Don't call plugin_call_cleanup. * plugin.c (plugin_call_cleanup): Make it static. (message): Don't call plugin_call_cleanup. (plugin_load_plugins): Call xatexit with plugin_call_cleanup. --- ld/ChangeLog | 10 ++++++++++ ld/ldmain.c | 7 ------- ld/plugin.c | 16 ++++++++++------ ld/plugin.h | 3 --- 4 files changed, 20 insertions(+), 16 deletions(-) diff --git a/ld/ChangeLog b/ld/ChangeLog index 55e320e98f3..3d200613bff 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,13 @@ +2010-12-06 H.J. Lu + Dmitry Gorbachev + + PR ld/12289 + * ldmain.c (main): Don't call plugin_call_cleanup. + + * plugin.c (plugin_call_cleanup): Make it static. + (message): Don't call plugin_call_cleanup. + (plugin_load_plugins): Call xatexit with plugin_call_cleanup. + 2010-12-05 H.J. Lu PR ld/12289 diff --git a/ld/ldmain.c b/ld/ldmain.c index c5b7d889ff8..5530dcaed1f 100644 --- a/ld/ldmain.c +++ b/ld/ldmain.c @@ -479,13 +479,6 @@ main (int argc, char **argv) lang_finish (); -#ifdef ENABLE_PLUGINS - /* Now everything is finished, we can tell the plugins to clean up. */ - if (plugin_call_cleanup ()) - info_msg (_("%P: %s: error in plugin cleanup (ignored)\n"), - plugin_error_plugin ()); -#endif /* ENABLE_PLUGINS */ - /* Even if we're producing relocatable output, some non-fatal errors should be reported in the exit status. (What non-fatal errors, if any, do we want to ignore for relocatable output?) */ diff --git a/ld/plugin.c b/ld/plugin.c index 3d7809c2c40..2950295b243 100644 --- a/ld/plugin.c +++ b/ld/plugin.c @@ -108,6 +108,9 @@ static bfd_boolean no_more_claiming = FALSE; TRUE is returned from the hook. */ static bfd_boolean plugin_cached_allow_multiple_defs = FALSE; +/* Call 'cleanup' hook for all plugins at exit. */ +static void plugin_call_cleanup (void); + /* List of tags to set in the constant leading part of the tv array. */ static const enum ld_plugin_tag tv_header_tags[] = { @@ -590,9 +593,6 @@ message (int level, const char *format, ...) char *newfmt = ACONCAT ((level == LDPL_FATAL ? "%P%F: " : "%P%X: ", format, "\n", NULL)); - if (plugin_call_cleanup ()) - info_msg (_("%P: %s: error in plugin cleanup (ignored)\n"), - plugin_error_plugin ()); vfinfo (stderr, newfmt, args, TRUE); } break; @@ -716,6 +716,8 @@ plugin_load_plugins (void) if (!curplug) return 0; + xatexit (plugin_call_cleanup); + /* First pass over plugins to find max # args needed so that we can size and allocate the tv array. */ while (curplug) @@ -812,8 +814,8 @@ plugin_call_all_symbols_read (void) return plugin_error_p () ? -1 : 0; } -/* Call 'cleanup' hook for all plugins. */ -int +/* Call 'cleanup' hook for all plugins at exit. */ +static void plugin_call_cleanup (void) { plugin_t *curplug = plugins_list; @@ -832,7 +834,9 @@ plugin_call_cleanup (void) } curplug = curplug->next; } - return plugin_error_p () ? -1 : 0; + if (plugin_error_p ()) + info_msg (_("%P: %s: error in plugin cleanup (ignored)\n"), + plugin_error_plugin ()); } /* Lazily init the non_ironly hash table. */ diff --git a/ld/plugin.h b/ld/plugin.h index e95c3eecf46..fc5f37f2dab 100644 --- a/ld/plugin.h +++ b/ld/plugin.h @@ -50,9 +50,6 @@ extern int plugin_call_claim_file (const struct ld_plugin_input_file *file, /* Call 'all symbols read' hook for all plugins. */ extern int plugin_call_all_symbols_read (void); -/* Call 'cleanup' hook for all plugins. */ -extern int plugin_call_cleanup (void); - /* Generate a dummy BFD to represent an IR file, for any callers of plugin_call_claim_file to use as the handle in the ld_plugin_input_file struct that they build to pass in. The BFD is initially writable, so -- 2.30.2