Hi Carl, On Wed, Mar 28, 2018 at 11:51:09AM -0700, Carl Love wrote: > gcc/ChangeLog: > > 2018-03-20 Carl Love <c...@us.ibm.com> > > Reverting patch: > 2017-09-27 Carl Love <c...@us.ibm.com> > > * config/rs6000/rs6000-builtin.def: Remove macro expansion for > FCTIW and FCTID. Remove macro definition BU_FP_MISC_1. > * config/rs6000/rs6000.md: Remove define_insn lrintsfsi2.
Please just say 2018-03-28 Carl Love <c...@us.ibm.com> Revert 2017-09-27 Carl Love <c...@us.ibm.com> * config/rs6000/rs6000-builtin.def (BU_FP_1MISC_1): Add define macro. (FCTID, FCTIW): Add BU_FP_MISC_1 macro expansion for builtins. * config/rs6000/rs6000.md (lrintsfsi2): Add define_insn for the fctiw instruction. i.e. the exact changelog of what you revert (or the part you revert, if the revert is partial). It's easier for people who want to find things in history, and easier for you to write too :-) Okay for trunk with that change (and the similar one for the testsuite changelog). Oh and don't forget "svn rm" :-) Thanks! Segher