On Tue, Oct 9, 2012 at 7:20 PM, Michael Meissner <meiss...@linux.vnet.ibm.com> wrote: > 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 <meiss...@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.
This patch is okay. Thanks, David
