Re: C++ PATCH to improve dump_decl (PR c++/71075)

OK. On Thu, May 19, 2016 at 4:03 PM, Marek Polacek wrote: > On Thu, May 19, 2016 at 07:55:05PM +0200, Marek Polacek wrote: >> On Thu, May 19, 2016 at 12:55:18PM -0400, Jason Merrill wrote: >> > Well, constants aren't declarations. Why are we calling dump_decl here? >> >> Eh,

Re: C++ PATCH to improve dump_decl (PR c++/71075)

On Thu, May 19, 2016 at 07:55:05PM +0200, Marek Polacek wrote: > On Thu, May 19, 2016 at 12:55:18PM -0400, Jason Merrill wrote: > > Well, constants aren't declarations. Why are we calling dump_decl here? > > Eh, not sure what I was thinking... > > We call dump_decl because

Re: C++ PATCH to improve dump_decl (PR c++/71075)

On Thu, May 19, 2016 at 12:55:18PM -0400, Jason Merrill wrote: > Well, constants aren't declarations. Why are we calling dump_decl here? Eh, not sure what I was thinking... We call dump_decl because unify_template_argument_mismatch has %qD: inform (input_location, " template

Re: C++ PATCH to improve dump_decl (PR c++/71075)

Well, constants aren't declarations. Why are we calling dump_decl here? Jason

C++ PATCH to improve dump_decl (PR c++/71075)

This PR compains about ugly diagnostics where we print template argument '2' does not match '#'integer_cst' not supported by dump_decl#' The following patch teaches dump_decl how to print constants properly. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2016-05-19 Marek Polacek