ogoffart abandoned this revision.
ogoffart added a comment.
Replaced by http://reviews.llvm.org/D19764
http://reviews.llvm.org/D19327
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ogoffart added a comment.
An alternative patch is uploaded there: http://reviews.llvm.org/D19764
http://reviews.llvm.org/D19327
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ogoffart added inline comments.
Comment at: lib/Sema/SemaDecl.cpp:5044-5045
@@ -5043,4 +5043,4 @@
// function template specialization, add it to the scope stack.
- if (New->getDeclName() && AddToScope &&
- !(D.isRedeclaration() && New->isInvalidDecl())) {
+ if
rsmith added inline comments.
Comment at: lib/Sema/SemaDecl.cpp:5044-5045
@@ -5043,4 +5043,4 @@
// function template specialization, add it to the scope stack.
- if (New->getDeclName() && AddToScope &&
- !(D.isRedeclaration() && New->isInvalidDecl())) {
+ if
ogoffart added a comment.
Ping?
http://reviews.llvm.org/D19327
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ogoffart created this revision.
ogoffart added reviewers: cfe-commits, rsmith.
struct XX {
double foo(invalid_type xx);
};
double XX::foo(invalid_type xx)
{
return 45;
}
We should keep XX::foo and its function body as part of the AST so tools can
still do something with the body even