Hi Pat, On Wed, Nov 08, 2017 at 10:59:23AM -0600, Pat Haugen wrote: > The following patch creates a new insn type to annotate quad precision > multiply instructions, updates the appropriate insns to use the new type > and creates an entry in the Power9 machine description which describes > the correct latency/resources. Bootstrap/regtest on powerpc64le-linux > with no new regressions. Ok for trunk?
One question. > --- gcc/config/rs6000/power9.md (revision 254377) > +++ gcc/config/rs6000/power9.md (working copy) > @@ -436,6 +436,12 @@ (define_insn_reservation "power9-qpdiv" > (eq_attr "cpu" "power9")) > "DU_super_power9,dfu_power9") > > +(define_insn_reservation "power9-qpmul" 24 > + (and (eq_attr "type" "qmul") > + (eq_attr "size" "128") > + (eq_attr "cpu" "power9")) > + "DU_super_power9,dfu_power9*12") All other p9 reservations (other than integer div) do not reserve a unit more than one cycle. Will this blow up size of the automaton? If not, should the other qp ops not reserve for more cycles, too? Rest looks fine; okay for trunk with the reservation thing taken care of one way or the other. Thanks! Segher