[cfe-commits] r162488 - /cfe/trunk/lib/AST/ASTContext.cpp

[Dmitri Gribenko]

Author: gribozavr
Date: Thu Aug 23 17:40:40 2012
New Revision: 162488

URL: http://llvm.org/viewvc/llvm-project?rev=162488&view=rev
Log:
Attaching comments to decls: since it was decided that Decl::isImplicit should
not be set for implicit instantiations, remove the FIXME.  This should be the
last bit for PR13634.  The actual fix happened in r162238.

Motivation: it might be misleading to mark implicit instantiations as
Decl::isImplicit = true.  Because then, in order to be consistent, we should
mark all instantiated members as implicit.  But the user did actually type the
declaration for the member, but the compiler played with it a little bit.

Modified:
    cfe/trunk/lib/AST/ASTContext.cpp

Modified: cfe/trunk/lib/AST/ASTContext.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ASTContext.cpp?rev=162488&r1=162487&r2=162488&view=diff
==============================================================================
--- cfe/trunk/lib/AST/ASTContext.cpp (original)
+++ cfe/trunk/lib/AST/ASTContext.cpp Thu Aug 23 17:40:40 2012
@@ -67,8 +67,6 @@
     return NULL;
 
   // User can not attach documentation to implicit instantiations.
-  // FIXME: all these implicit instantiations shoud be marked as implicit
-  // declarations and get caught by condition above.
   if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(D)) {
     if (FD->getTemplateSpecializationKind() == TSK_ImplicitInstantiation)
       return NULL;