Re: [PATCH] Rs6000 infrastructure cleanup (switches), revised patch #2e

[Michael Meissner <meissner at linux dot vnet dot ibm dot com>]

This patch adds more debugging via -mdebug=reg to the compiler, and it is the
main way I verified that all of the options were set correctly.  If you do not
use -mdebug=reg, this patch has no effect.  When I emailed this patch, I had
bootstraped the compiler, and I was beginning to do make check.  Assuming there
are no regressions, is it ok to check in?

2012-10-09  Michael Meissner  <meissner@linux.vnet.ibm.com>

	* config/rs6000/rs6000.c (DEBUG_FMT_ID): Move "-32s" to a separate
	define and change DEBUG_FMT_<x> to use it.
	(DEBUG_FMT_D): Likewise.
	(DEBUG_FMT_S): Likewise.
	(DEBUG_FMT_X): Delete, no longer used.
	(DEBUG_FMT_W): Likewise.
	(DEBUG_FMT_WX): New debug format for printing options in a
	friendly fashion.
	(rs6000_debug_reg_global): If -mdebug=reg, print all of the
	options in target_flags and target_flags_explicit.  Print the
	default options for -mcpu=<xxx>, -mtune=<xxx>, and the default
	options.  Adjust printing out the builtin options.
	(rs6000_option_override_internal): Change printing the builtin
	options to use rs6000_print_builtin_options.
	(rs6000_function_specific_print): Change to use
	rs6000_print_isa_options to print ISA flags.
	(rs6000_print_options_internal): New function for expanded
	-mdebug=reg option printing to print both the ISA options, and the
	builtins that are enabled.
	(rs6000_print_isa_options): New function to print the ISA
	options.
	(rs6000_print_builtin_options): New function to print the builtin
	functions enabled.

-- 
Michael Meissner, IBM
5 Technology Place Drive, M/S 2757, Westford, MA 01886-3141, USA
meissner@linux.vnet.ibm.com	fax +1 (978) 399-6899

Attachment: gcc-power7.patch361b
Description: Text document