On Wed, Mar 21, 2018 at 12:47:41PM -0500, Peter Bergner wrote: > On 3/20/18 12:27 PM, Peter Bergner wrote: > > On 3/20/18 11:42 AM, Segher Boessenkool wrote: > >> On Mon, Mar 19, 2018 at 09:12:08PM -0500, Peter Bergner wrote: > >>> Looking at mu build dirs insn-modes.h, I don't see HAVE_V8HFmode being > >>> defined on either my LE or BE builds. What am I missing? > >> > >> Nothing, I just was confused (we always define the mode in > >> rs6000-modes.def, > >> but that is not the same thing). Sorry. > > > > Ok then, patch committed with the expander change you requested and > > leaving the HAVE_V8HFmode code as is in the patch. Thanks! > > Kaushik confirmed this is broken on GCC 7 as well. Ok to backport > this patch to GCC 7, assuming regtesting is clean? > > I'll note that GCC 7 does not need any of the changes to rs6000-p8swap.c, > since that file doesn't exist and doesn't need to exist in GCC 7, so I've > left those changes out.
Did that same code live in rs6000.c in GCC 7, or is it new since rs6000-p8swap.c was split off from there? And what about GCC 6? Segher
