On 21 March 2017 at 02:36, Ville Voutilainen <ville.voutilai...@gmail.com> wrote: > On 21 March 2017 at 01:44, Jason Merrill <ja...@redhat.com> wrote: >>> It looks strange to me. Why not change the definition of check_new instead >>> of changing the condition that uses it? >> >> Agreed. Also, let's factor the new tests out into a function, say >> non_allocating_fn_p. > > This also conveniently let's me provide a ChangeLog that reads like a > Real Front-End > Developer would've written it. :) New tests re-tested on Linux-x64, > running full suite > on Linux-PPC64.
Hmm. I should either rename that function or flip its logic. Now it's a bit backwards. :) I'll flip its logic.