Hi! On Tue, Dec 08, 2020 at 03:46:31PM -0600, Pat Haugen wrote: > gcc/ > * config/rs6000/dfp.md (extendddtd2, trunctddd2, *cmp<mode>_internal1, > floatditd2, ftrunc<mode>2, fix<mode>di2, dfp_ddedpd_<mode>, > dfp_denbcd_<mode>, dfp_dxex_<mode>, dfp_diex_<mode>, > *dfp_sgnfcnc_<mode>, dfp_dscli_<mode>, dfp_dscri_<mode>): Update size > attribute for Power10. > * config/rs6000/mma.md (*movoo): Likewise. > * config/rs6000/rs6000.md (define_attr "size"): Add 256. > (define_mode_attr bits): Add DD/TD modes. > * config/rs6000/sync.md (load_quadpti, store_quadpti, load_lockedpti, > store_conditionalpti): Update size attribute for Power10.
> --- a/gcc/config/rs6000/mma.md > +++ b/gcc/config/rs6000/mma.md > @@ -288,6 +288,7 @@ (define_insn_and_split "*movoo" > DONE; > } > [(set_attr "type" "vecload,vecstore,veclogical") > + (set_attr "size" "256,256,*") > (set_attr "length" "*,*,8")]) Does "*" do the right thing here? It has "32" as the default, is that wanted? It doesn't matter much at all of course, but I'd expect 128 or even just 256 for all alternatives. Okay for trunk with or without that tweaked. Thanks! Segher
