On Wed, Oct 16, 2019 at 5:06 PM H.J. Lu <hjl.to...@gmail.com> wrote: > > i386.h has > > #define CLEAR_RATIO(speed) ((speed) ? MIN (6, ix86_cost->move_ratio) : 2) > > It is impossible to have CLEAR_RATIO > 6. This patch adds clear_ratio > to processor_costs, sets it to the minimum of 6 and move_ratio in all > cost models and defines CLEAR_RATIO with clear_ratio. > > * config/i386/i386.h (processor_costs): Add clear_ratio. > (CLEAR_RATIO): Remove MIN and use ix86_cost->clear_ratio. > * config/i386/x86-tune-costs.h: Set clear_ratio to the minimum > of 6 and move_ratio in all cost models. > > OK for trunk?
LGTM. Are these numbers backed by some benchmark results? Thanks, Uros. > > -- > H.J.
