On Fri, May 13, 2016 at 08:23:16PM -0500, Segher Boessenkool wrote: > On Fri, May 13, 2016 at 07:25:43PM -0400, Michael Meissner wrote: > > This patch adds support for the 32-bit word splat instructions, the byte > > immediate splat instructions, and the vector sign extend instructions to GCC > > 7.0. > > > > In addition to the various splat instructions, since I was modifying the > > vector > > moves, I took the opportunity to reorganize the vector move instructions > > with > > several changes I've wanted to do: > > It is much easier to review, and for regression searches later, if one > patch does one thing. No need to change this patch, but please keep > it in mind for later patches.
Yes and no. Sometimes, you need to make larger changes. > > Are these patches ok to apply to the GCC 7.0 trunk? > > Changelog is missing. Sorry about that. I was focused on including the patches this time, that I forgot to include the ChangeLog entries. Unfortunately, I'm finding some regressions that seem to show up with more recent trunk changes, particularly with reload than LRA. I'm going to have to spend some time debugging why reload is failing on big endian systems, and resubmit the patches when it is debugged. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797
