On Wednesday 15 January 2014 21:55:05 Richard Smith wrote: > On Wed Jan 15 2014 at 1:51:41 PM, Justin Bogner <[email protected]> > > wrote: > > Olivier Goffart <[email protected]> writes: > > > From d131af55d2dc2cb6b2f7242697eb44d2e5d042da Mon Sep 17 00:00:00 2001 > > > From: Olivier Goffart <[email protected]> > > > Date: Wed, 15 Jan 2014 19:45:12 +0100 > > > Subject: [PATCH] Fix source range of the destructor name. > > > > > > The problem is that the destructor's DeclarationNameInfo do not have > > > a TypeSourceInfo because Sema::GetNameForDeclarator requires the > > > ParsedType to be a LocInfoType. > > > > > > Setting a proper TypeSourceInfo to the destructor changes the way it > > > it printed (from '~Foo' to '~struct Foo'. Hence the change in > > > DeclarationName.cpp which also fix a bug when printing operator names. > > > > > > http://llvm.org/bugs/show_bug.cgi?id=15125 > > > > Please add a test with this. Some minor comments below.
Thanks for the comments. Any hint on how the test should be? Is there already existing tests which tests source locations? Thanks -- Olivier _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
