Re: [PATCH ARM]Print CPU tuning information as comment in assembler file.
Ping. This is for case failures and it doesn't affect normal compilation, so I suppose it's fine for this stage? Thanks, bin On Fri, Mar 6, 2015 at 3:42 PM, Bin Cheng bin.ch...@arm.com wrote: Hi, This patch is the first part fixing memset-inline-{4,5,6,8,9}.c failures on cortex-a9. GCC/arm doesn't generate any tuning information in assembly, it can't tell whether we are compiling for cortex-a9 tune if the compiler is configured so by default. This patch introduces a new (target dependent) option -mprint-tune-info. It prints CPU tuning information as comment in assembler file, thus DEJAGNU can check it and make decisions. By default the option is disabled, so it won't change current behaviors. For now, pointers in tune structure are not printed, we should improve that and output more useful information in the long run. Another patch is followed adding DEJAGNU test function and adapting test strings. Build and test on arm-none-eabi, is it OK? 2015-03-06 Bin Cheng bin.ch...@arm.com * config/arm/arm.opt (print_tune_info): New option. * config/arm/arm.c (arm_print_tune_info): New function. (arm_file_start): Call arm_print_tune_info. * config/arm/arm-protos.h (struct tune_params): Add comment. * doc/invoke.texi (@item -mprint-tune-info): New item. (-mtune): mention it in ARM Option Summary.
Re: [PATCH ARM]Print CPU tuning information as comment in assembler file.
On Fri, Mar 6, 2015 at 7:42 AM, Bin Cheng bin.ch...@arm.com wrote: Hi, This patch is the first part fixing memset-inline-{4,5,6,8,9}.c failures on cortex-a9. GCC/arm doesn't generate any tuning information in assembly, it can't tell whether we are compiling for cortex-a9 tune if the compiler is configured so by default. This patch introduces a new (target dependent) option -mprint-tune-info. It prints CPU tuning information as comment in assembler file, thus DEJAGNU can check it and make decisions. By default the option is disabled, so it won't change current behaviors. For now, pointers in tune structure are not printed, we should improve that and output more useful information in the long run. Another patch is followed adding DEJAGNU test function and adapting test strings. Build and test on arm-none-eabi, is it OK? This is OK thanks. Ramana 2015-03-06 Bin Cheng bin.ch...@arm.com * config/arm/arm.opt (print_tune_info): New option. * config/arm/arm.c (arm_print_tune_info): New function. (arm_file_start): Call arm_print_tune_info. * config/arm/arm-protos.h (struct tune_params): Add comment. * doc/invoke.texi (@item -mprint-tune-info): New item. (-mtune): mention it in ARM Option Summary.