Some cleanup of TODO and REVIEW comments.

Author: mattmccutchen-cci

clang/include/clang/3C/MultiDecls.h

@@ -153,8 +153,6 @@ struct MultiDeclInfo {
   // Set by DeclRewriter::rewriteMultiDecl after it rewrites the entire
   // multi-decl to ensure that it doesn't try to do so more than once if
   // multiple members needed changes.
-  // REVIEW: A design argument could be made that this flag doesn't belong here
-  // and the rewriter should instead keep a visited set or something like that.
   bool AlreadyRewritten = false;
 };
 
@@ -179,7 +177,7 @@ class ProgramMultiDeclsInfo {
   // Set of TagDecl names already used at least once in the program, so we can
   // avoid colliding with them.
   std::set<std::string> UsedTagNames;
-  
+
   // Information about an originally unnamed tag definition in a multi-decl for
   // which we're using a new name.
   struct RenamedTagDefInfo {

clang/lib/3C/DeclRewriter.cpp

@@ -404,10 +404,10 @@ void DeclRewriter::rewriteMultiDecl(MultiDeclInfo &MDI, RSet &ToRewrite) {
       // thinking that they can replace a different source range that way, we
       // want to fail fast.
       //
-      // REVIEW: This is awkward and makes me wonder if we should just remove
+      // This is awkward and makes me wonder if we should just remove
       // DeclReplacement::getSourceRange since 3C currently only calls
       // getSourceRange on an object already known to be a
-      // FunctionDeclReplacement. But after drafting this, I wasn't convinced
+      // FunctionDeclReplacement. But after drafting that, I wasn't convinced
       // that it was better than the status quo.
       assert(Replacement->getSourceRange(SM) == ReplaceSR);
     }

clang/lib/3C/MultiDecls.cpp

@@ -10,7 +10,7 @@
 #include "clang/3C/Utils.h"
 
 MultiDeclMemberDecl *getAsMultiDeclMember(Decl *D) {
-  // REVIEW: Is this the best place for this check?
+  // XXX: Is this the best place for this check?
   if (D->getLocation().isInvalid())
     return nullptr;
 

clang/test/3C/inline_anon_structs.c

@@ -182,7 +182,6 @@ void foo2(int *x) {
 
 // Tests of new functionality from
 // https://github.com/correctcomputation/checkedc-clang/pull/657.
-// TODO: Do we want to put this in a separate file?
 
 // Test that 3C doesn't mangle the code by attempting to split a forward
 // declaration of a struct out of a multi-decl as if it were a definition

clang/test/3C/multivardecls.c

@@ -267,7 +267,6 @@ _Array_ptr<int> p1 : bounds((0, lo), hi), p2 : bounds(lo, (0, hi)), *p3;
 // https://github.com/correctcomputation/checkedc-clang/issues/651.
 // inline_anon_structs.c has a few additional tests of typedef multi-decls
 // involving inline structs.
-// TODO: Are there other cases we should test?
 
 typedef int *A, *B;
 // CHECK: typedef _Ptr<int> A;