On Mon, Jul 12, 2021 at 12:47:43PM -0500, Bill Schmidt wrote:
> On 7/12/21 12:16 PM, Michael Meissner wrote:
> >>Shouldn't this last be { lp64 && has_arch_pwr10 } ?
> >Nope. Because the power10 vector multiply is done in the vector unit, it
> >can
> >generate the vmulld instruction.
>
> Please document this, then.
And this really should not use lp64 but powerpc64, *but* that test is
broken still (it does not test if the *compiler* allows generating
these insns, *only* if the machine can execute the insns, and it is
otherwise broken on AIX, etc.) We really should impprove that :-(
If it had used powerpc64 here, there would not have been this confusion.
The problem in fixing this of course is the historical uses of this
test. Bah.
Segher
