Commit 4c3784ae by Georg-Johann Lay Committed by Georg-Johann Lay

Fix breakage introduced by r276985.

	* config/avr/avr.c (avr_option_override): Remove set of
	PARAM_ALLOW_STORE_DATA_RACES.
	* common/config/avr/avr-common.c (avr_option_optimization_table)
	[OPT_LEVELS_ALL]: Turn on -fallow-store-data-races.

From-SVN: r277115
parent 25e22b19
2019-10-17 Georg-Johann Lay <avr@gjlay.de>
Fix breakage introduced by r276985.
* config/avr/avr.c (avr_option_override): Remove set of
PARAM_ALLOW_STORE_DATA_RACES.
* common/config/avr/avr-common.c (avr_option_optimization_table)
[OPT_LEVELS_ALL]: Turn on -fallow-store-data-races.
2019-10-17 H.J. Lu <hongjiu.lu@intel.com> 2019-10-17 H.J. Lu <hongjiu.lu@intel.com>
* config/i386/i386.h (processor_costs): Add clear_ratio. * config/i386/i386.h (processor_costs): Add clear_ratio.
......
...@@ -38,6 +38,11 @@ static const struct default_options avr_option_optimization_table[] = ...@@ -38,6 +38,11 @@ static const struct default_options avr_option_optimization_table[] =
{ OPT_LEVELS_ALL, OPT_fcaller_saves, NULL, 0 }, { OPT_LEVELS_ALL, OPT_fcaller_saves, NULL, 0 },
{ OPT_LEVELS_1_PLUS_NOT_DEBUG, OPT_mgas_isr_prologues, NULL, 1 }, { OPT_LEVELS_1_PLUS_NOT_DEBUG, OPT_mgas_isr_prologues, NULL, 1 },
{ OPT_LEVELS_1_PLUS, OPT_mmain_is_OS_task, NULL, 1 }, { OPT_LEVELS_1_PLUS, OPT_mmain_is_OS_task, NULL, 1 },
/* Allow optimizer to introduce store data races. This used to be the
default -- it was changed because bigger targets did not see any
performance decrease. For the AVR though, disallowing data races
introduces additional code in LIM and increases reg pressure. */
{ OPT_LEVELS_ALL, OPT_fallow_store_data_races, NULL, 1 },
{ OPT_LEVELS_NONE, 0, NULL, 0 } { OPT_LEVELS_NONE, 0, NULL, 0 }
}; };
......
...@@ -741,15 +741,6 @@ avr_option_override (void) ...@@ -741,15 +741,6 @@ avr_option_override (void)
if (avr_strict_X) if (avr_strict_X)
flag_caller_saves = 0; flag_caller_saves = 0;
/* Allow optimizer to introduce store data races. This used to be the
default - it was changed because bigger targets did not see any
performance decrease. For the AVR though, disallowing data races
introduces additional code in LIM and increases reg pressure. */
maybe_set_param_value (PARAM_ALLOW_STORE_DATA_RACES, 1,
global_options.x_param_values,
global_options_set.x_param_values);
/* Unwind tables currently require a frame pointer for correctness, /* Unwind tables currently require a frame pointer for correctness,
see toplev.c:process_options(). */ see toplev.c:process_options(). */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment