[PATCH] Fix/Improve SourceRange of defaulted destructors

Richard Smith richard at metafoo.co.uk
Wed Jun 18 01:05:24 PDT 2014


================
Comment at: lib/Sema/SemaDeclCXX.cpp:8883
@@ -8882,3 +8882,3 @@
 
-  SourceLocation Loc = Destructor->getLocation();
+  SourceLocation Loc = Destructor->getLocEnd();
   Destructor->setBody(new (Context) CompoundStmt(Loc));
----------------
I think this is a reasonable approach. Do the other `DefineImplicit*` members also need this fix?

http://reviews.llvm.org/D4175






More information about the cfe-commits mailing list