On Mon, Apr 24, 2017 at 05:38:58PM -0500, Bill Seurer wrote: > [PATCH, rs6000] pr80482 Relax vector builtin parameter checks > > This patch changes the parameter testing for powerpc vector builtins to relax > the existing requirement that the parameters be identical to instead that they > be compatible. This allows for mixing parameters with differing qualified > (const, volatile, etc.) types. > > See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80482 for more information. > > Bootstrapped and tested on powerpc64le-unknown-linux-gnu and > powerpc64be-unknown-linux-gnu with no regressions. Is this ok for trunk?
It looks fine to me, okay for trunk, thanks (with Jakub's comment taken care of). Also okay for the 7 branch if the RMs agree (it fixes a regression from GCC 6 and it seems unlikely to cause new problems). Segher > [gcc] > > 2017-04-24 Bill Seurer <seu...@linux.vnet.ibm.com> > > * config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin): Change > type checks to test for compatibility instead of equality. > > [gcc/testsuite] > > 2017-04-24 Bill Seurer <seu...@linux.vnet.ibm.com> > > * gcc.target/powerpc/vec-constvolatile.c: New test.
