> On Jul 8, 2016, at 12:14 AM, Segher Boessenkool <seg...@kernel.crashing.org> > wrote: > > 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.
No backports required; this is a 7 regression. Bill > > > 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 (). >
