[PATCH] Fix source range of defaulted/deleted member declarations

Eli Bendersky eliben at google.com
Thu Mar 19 14:35:09 PDT 2015


Hi rnk, jpienaar,

Fixes https://llvm.org/bugs/show_bug.cgi?id=20744

struct A {                                                                                                                            
  A() = default;
};

Previously the source range of the declaration of A ended at the ')'. It should include the '= default' part as well. The same for '= delete'.

Note: this will break one of the clang-tidy fixers, which is going to be addessed in a follow-up patch.

http://reviews.llvm.org/D8465

Files:
  lib/Parse/ParseDeclCXX.cpp
  test/Analysis/inlining/path-notes.cpp
  test/Misc/ast-dump-decl.cpp
  unittests/AST/SourceLocationTest.cpp

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8465.22303.patch
Type: text/x-patch
Size: 5941 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150319/29155f79/attachment.bin>


More information about the cfe-commits mailing list