From patchwork Wed Apr 6 07:44:52 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: fix PR bootstrap/48471 Date: Tue, 05 Apr 2011 21:44:52 -0000 From: Tristan Gingold X-Patchwork-Id: 89993 Message-Id: <8A64F4A5-F1F5-48D9-B3EF-CFB63546D531@adacore.com> To: "gcc-patches@gcc.gnu.org Patches" Cc: Nathan Froyd On Apr 6, 2011, at 3:06 AM, Nathan Froyd wrote: > My changes that moved bits of final.c to dbxout.c caused compilation > failures on targets that *didn't* enable DBX_DEBUGGING_INFO, since some > now-static symbols could be unused. To fix this, the patch below moves > all the new static variables and two macros that use some of said > variables below an #ifdef DBX_DEBUGGING_INFO. Steve Ellcey verified > that the patch fixes the problems on ia64-hp-hpux11.23 (thanks Steve!) > and I verified that the changes still bootstrap on > x86_64-unknown-linux-gnu. > > Committed as obvious. Sorry about all the build breakage today, not one > of my better cleanup days... Nathan, bootstrap still fails for xcoff targets (eg aix). I can fix the failure with this patch. Ok for trunk ? Tristan. 2011-04-06 Tristan Gingold * dbxout.c: (debug_nesting, symbol_queue, symbol_queue_index) (symbol_queue_size, DBXOUT_DECR_NESTING) (DBXOUT_DECR_NESTING_AND_RETURN): Also define if XCOFF_DEBUGGING_INFO. --- a/gcc/dbxout.c +++ b/gcc/dbxout.c @@ -903,7 +903,7 @@ dbxout_finish_complex_stabs (tree sym, stab_code_type code, obstack_free (&stabstr_ob, str); } -#if defined (DBX_DEBUGGING_INFO) +#if defined (DBX_DEBUGGING_INFO) || defined (XCOFF_DEBUGGING_INFO) /* When -gused is used, emit debug info for only used symbols. But in addition to the standard intercepted debug_hooks there are some @@ -926,6 +926,10 @@ static int symbol_queue_size = 0; #define DBXOUT_DECR_NESTING_AND_RETURN(x) \ do {--debug_nesting; return (x);} while (0) +#endif /* DBX_DEBUGGING_INFO || XCOFF_DEBUGGING_INFO */ + +#if defined (DBX_DEBUGGING_INFO) + static void dbxout_function_end (tree decl ATTRIBUTE_UNUSED) {