On Thu, Oct 17, 2019 at 8:47 AM Uros Bizjak <ubiz...@gmail.com> wrote: > > 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?
Note CLEAR_RATIO is also used by simplification of static initializers and thus changing it can have "interesting" effects (on the testsuite). Richard. > Thanks, > Uros. > > > > > -- > > H.J.