r215375 - Add some FIXMEs.

Richard Smith richard-llvm at metafoo.co.uk
Mon Aug 11 11:35:44 PDT 2014


Author: rsmith
Date: Mon Aug 11 13:35:44 2014
New Revision: 215375

URL: http://llvm.org/viewvc/llvm-project?rev=215375&view=rev
Log:
Add some FIXMEs.

Modified:
    cfe/trunk/lib/AST/Decl.cpp
    cfe/trunk/lib/Serialization/ASTWriter.cpp

Modified: cfe/trunk/lib/AST/Decl.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/Decl.cpp?rev=215375&r1=215374&r2=215375&view=diff
==============================================================================
--- cfe/trunk/lib/AST/Decl.cpp (original)
+++ cfe/trunk/lib/AST/Decl.cpp Mon Aug 11 13:35:44 2014
@@ -811,6 +811,8 @@ static LinkageInfo getLVForNamespaceScop
 
   // Everything not covered here has no linkage.
   } else {
+    // FIXME: A typedef declaration has linkage if it gives a type a name for
+    // linkage purposes.
     return LinkageInfo::none();
   }
 

Modified: cfe/trunk/lib/Serialization/ASTWriter.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Serialization/ASTWriter.cpp?rev=215375&r1=215374&r2=215375&view=diff
==============================================================================
--- cfe/trunk/lib/Serialization/ASTWriter.cpp (original)
+++ cfe/trunk/lib/Serialization/ASTWriter.cpp Mon Aug 11 13:35:44 2014
@@ -3829,6 +3829,8 @@ void ASTWriter::WriteRedeclarations() {
           FirstFromAST = Prev;
       }
 
+      // FIXME: Do we need to do this for the first declaration from each
+      // redeclaration chain that was merged into this one?
       Chain->MergedDecls[FirstFromAST].push_back(getDeclID(First));
     }
 





More information about the cfe-commits mailing list