On Fri, Apr 07, 2017 at 11:58:15AM -0500, Pat Haugen wrote:
> The following patch changes the method of scheduling vector and vector
> load insns. Before it tried to pair up like insns and interleave the
> pairs, resulting in something like L1L2V1V2. The preferred scheduling is
> now to just interleave the insns, resulting in L1V1L2V2. If interleaving
> fails, fall back to pairing like insns.

> +          1  : A vecload or vector insn has been issued and a candidate for
> +               for pairing has been found and moved to the end of the ready

Duplicate word ("for for").

> +                   if (is_power9_pairable_vec_type (get_attr_type
> +                                                    (ready[pos])))

A good way to battle long lines is to introduce another local var.

Okay for trunk and branch (with the duplicate word fixed, and do
whatever you think best with the long line).  Thanks,


Segher

Reply via email to