On Wed, Apr 24, 2019 at 5:26 PM Marek Polacek <pola...@redhat.com> wrote: > > This valid test started to be accepted with r265789, which removed the > following checks.
It didn't remove them, it moved them into invalid_tparm_referent_p. And changed the tests for C++17. > I wonder if we want to remove them in gcc 8, or just leave them in place. I wouldn't mess with GCC 8; that's a big patch, and it's not a regression, it's a C++17 change we didn't implement before. Jason