On Fri, Jan 13, 2017 at 02:58:23PM -0800, Carl E. Love wrote: > The following patch fixes an issue with the builtin test test_ne_long(). > The issue is the arguments need to be "bool long long" not "bool long" > to get the correct 64-bit value when running in 32-bit mode. > > The patch has been tested on: > powerpc64-unknown-linux-gnu (Power 7 64-bit, 32-bit) with no > regressions. > > Is this OK for trunk?
> 2017-01-13 Carl Love <c...@us.ibm.com> > > * gcc.target/powerpc/builtins-3-p9.c (test_ne_long()): > Test fails in 32-bit mode because the arguments are of type > "bool long" not "bool long long". The arguments need to be > "bool long long" to get a correct 64-bit value in 32-bit mode. Just "Change argument and return types to bool long long."? Okay for trunk with that (you didn't mention the return type before; the rest is fine if overly verbose, a changelog entry does not usually say "why", just "what"). Segher
