Hi Carl,
On Wed, Mar 14, 2018 at 08:32:54AM -0700, Carl Love wrote:
> 2018-03-13 Carl Love <[email protected]>
>
> PR 84422 - ICE on various builtin test functions when compiled with
> -mcpu=power7.
This should be _exactly_
PR target/84422
(one such line per PR, and no blank line after it) so that bugzilla
will automagically pick it up.
> * config/rs6000/rs6000-builtin.def: Change expansion for
> VMULE[SW|UW] to BU_P8V_AV_2.
> * config/rs6000/rs6000.c: Change
> ALTIVEC_BUILTIN_VMULE[SW|UW] to P8V_BUILTIN_VMULE[SW|UW].
> Change ALTIVEC_BUILTIN_VMULO[SW|UW] to P8V_BUILTIN_VMULO[SW|UW].
> * config/rs6000/rs6000-c.c: Change
> ALTIVEC_BUILTIN_VMULE[SW|UW] to P8V_BUILTIN_VMULE[SW|UW].
> Change ALTIVEC_BUILTIN_VMULO[SW|UW] to P8V_BUILTIN_VMULO[SW|UW].
If you don't use the literal names it becomes very hard to grep for.
The patch itself looks fine. Okay for trunk. Thanks!
Segher