On 9/12/18 6:44 AM, Martin Liška wrote: > Hi. > > Honza asked me to do the follow-up. It moves definition of alignments > related to a CPU into *_cost scructures. Advantage of it is that there's > no redundant definition for CPUs that have equal cost. > > I verified that it produces same output for all valid -march options: > gcc --help=common -Q -O2 -march=* > > I need to reg&bootstrap the patch. Will it be fine after I'll test it? > Thanks, > Martin > > gcc/ChangeLog: > > 2018-09-12 Martin Liska <mli...@suse.cz> > > * common/config/i386/i386-common.c (ix86_get_valid_option_values): > Use processor_names table. > * config/i386/i386.c (ix86_default_align): Use > processor_cost_table for alignment values. > (ix86_option_override_internal): Use processor_names. > (ix86_function_specific_print): Likewise. > * config/i386/i386.h (struct processor_costs): > Add alignment values. > (struct ptt): Remove and replace with const char *. > * config/i386/x86-tune-costs.h (struct processor_costs): > Declare default alignments for all costs. Ok after suitable testing.
jeff
