On Wed, 12 Jul 2023, Richard Biener wrote:
> > > That said, we should handle this better so can you file an
> > > enhancement bugreport for this?
> >
> > Filed as PR -optimization/110630.
>
> Thanks!
Thanks for making this improvement. I've checked MIPS results and code
produced now is as follows:
daddiu $sp,$sp,-64
sd $5,24($sp)
sd $7,40($sp)
ldc1 $f0,24($sp)
ldc1 $f1,40($sp)
sd $4,16($sp)
sd $6,32($sp)
ldc1 $f2,32($sp)
add.ps $f1,$f0,$f1
ldc1 $f0,16($sp)
add.ps $f0,$f0,$f2
sdc1 $f1,56($sp)
ld $3,56($sp)
sdc1 $f0,48($sp)
ld $2,48($sp)
jr $31
daddiu $sp,$sp,64
which does do vector stuff now, although it's still considerably worse
than my handwritten example:
> > dmtc1 $4,$f0
> > dmtc1 $5,$f1
> > dmtc1 $6,$f2
> > dmtc1 $7,$f3
> > add.ps $f0,$f0,$f1
> > add.ps $f2,$f2,$f3
> > dmfc1 $2,$f0
> > jr $31
> > dmfc1 $3,$f2
Or I'd say it's pretty terrible, but given the current situation with the
MIPS backend I'm going to leave it to the new maintainer to sort out.
> > Do you agree it still makes sense to include bb-slp-pr95839-v8.c with the
> > testsuite?
>
> Sure, more coverage is always nice.
Thanks, committed (with the `vect64' requirement removed, as we can take
it for granted with `vect_float').
Maciej
