> 2011-03-13 Joseph Myers <jos...@codesourcery.com> > > * config/sparc/sparc-opts.h: New.
Do you really need all the dates in there? > * config/sparc/sparc.c (sparc_handle_option, sparc_select, > sparc_cpu, fpu_option_set, TARGET_HANDLE_OPTION): Remove. > (sparc_option_override): Store processor_type enumeration rather > than string in cpu_default. Remove name and enumeration from > cpu_table. Directly default -mcpu then default -mtune from -mcpu > without using sparc_select. Use target_flags_explicit instead of > fpu_option_set. > * config/sparc/sparc.h (enum processor_type): Move to > sparc-opts.h. > (sparc_cpu, struct sparc_cpu_select, sparc_select): Remove. > * config/sparc/sparc.opt (config/sparc/sparc-opts.h): New > HeaderInclude entry. > (mcpu=, mtune=): Use Var and Enum. > (sparc_processor_type): New Enum and EnumValue entries. The sparc_cpu/sparc_cpu_option naming is confusing. What about replacing the latter with sparc_cpu_and_features or something along these lines? Otherwise looks good, thanks. -- Eric Botcazou