On Tue, 2020-12-08 at 15:46 -0600, Pat Haugen via Gcc-patches wrote: > Update size attribute for Power10. > > > This patch was broken out from my larger patch to update various > attributes for > Power10, in order to make the review process hopefully easier. This > patch only > updates the size attribute for various new instructions. There were > no changes > requested to this portion of the original patch, so nothing is new > here. > > Bootstrap/regtest on powerpc64le (Power8/Power10) with no new > regressions. Ok for trunk? > > -Pat > > > 2020-11-08 Pat Haugen <pthau...@linux.ibm.com> > > gcc/
I think you'll need to specify gcc/ChangeLog at commit time. Beyond that nit, the Changelog content here looks to match the patch body OK. lgtm, thanks -Will > * 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. >
