Re: [PATCH ARM]Print CPU tuning information as comment in assembler file.

2015-03-13 Thread Bin.Cheng
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.

2015-03-13 Thread Ramana Radhakrishnan
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.