On Thu, Jul 07, 2016 at 03:40:28PM -0500, Bill Schmidt wrote:
> > PR71297 reports that we ICE when __builtin_vec_ld or __builtin_vec_st is
> > provided with an incorrect number of arguments.  This patch fixes it by
> > bypassing special handling for these intrinsics when the number of
> > arguments is wrong, thus allowing the standard error handling for
> > builtins to kick in.
> > 
> > The patch is pretty obvious and I think adding a test case would be
> > extraneous, though I can do so if desired.

Well you could use the one from the PR?

> > Bootstrapped and tested on
> > powerpc64le-unknown-linux-gnu with no regressions, and the original
> > failure is fixed.  Is this ok for trunk?

Yes, but please do a testcase.  Okay for backports, too.


Segher


> >     PR target/71297
> >     * config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin):
> >     Allow standard error handling to take over when a wrong number
> >     of arguments is presented to __builtin_vec_ld () or
> >     __builtin_vec_st ().

Reply via email to