[PATCH][C++] Fix PR29433, make C++ use a lot less time/memory

[Mark Mitchell]

Jason Merrill wrote:

> I guess that we were missing this diagnostic before by accident, because
> of the different DECL_NAMEs for class template instantiations.  If we
> want to avoid the memory usage for different DECL_NAMEs, I'd think you'd
> have to hack around a lot to avoid giving this (correct) diagnostic here.

I don't think it would be too bad, but ...

> My inclination would be to make the change, and downgrade the error to
> pedwarn.  There's no need for it to be a hard error.

... that's a really good idea, since it gives users an escape hatch, but
costs us nearly zero.

Richard, would you please make that change as part of your patch?

Thanks,

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713