Add periods to the end of comments in 3C code.

Also fix some typos I noticed.

Author: mattmccutchen-cci

clang-tools-extra/clangd/tool/CMakeLists.txt

@@ -1,7 +1,7 @@
 include_directories(${CMAKE_CURRENT_SOURCE_DIR}/..)
 include_directories(${CMAKE_CURRENT_BINARY_DIR}/..)
 
-# Same idea as add_clangDaemon_target in ../CMakeLists.txt
+# Same idea as add_clangDaemon_target in ../CMakeLists.txt.
 function(add_clangd_target_part1 target_name)
 add_clang_tool(${target_name}
   ClangdMain.cpp

clang/include/clang/3C/CastPlacement.h

@@ -48,11 +48,11 @@ class CastPlacementVisitor : public RecursiveASTVisitor<CastPlacementVisitor> {
   ArrayBoundsRewriter ABRewriter;
   std::set<Expr *> &ExprsWithCast;
 
-  // Enumeration indicating what type of cast is required at a call site
+  // Enumeration indicating what type of cast is required at a call site.
   enum CastNeeded {
-    NO_CAST = 0,     // No casting required
-    CAST_TO_CHECKED, // A CheckedC bounds cast required (wild -> checked)
-    CAST_TO_WILD     // A standard C explicit cast required (checked -> wild)
+    NO_CAST = 0,     // No casting required.
+    CAST_TO_CHECKED, // A CheckedC bounds cast required (wild -> checked).
+    CAST_TO_WILD     // A standard C explicit cast required (checked -> wild).
   };
 
   CastNeeded needCasting(ConstraintVariable *SrcInt, ConstraintVariable *SrcExt,

clang/include/clang/3C/ConstraintResolver.h

@@ -32,7 +32,7 @@ class ConstraintResolver {
 
   // Returns a set of ConstraintVariables which represent the result of
   // evaluating the expression E. Will explore E recursively, but will
-  // ignore parts of it that do not contribute to the final result
+  // ignore parts of it that do not contribute to the final result.
   CVarSet getExprConstraintVars(Expr *E);
 
   CVarSet getCalleeConstraintVars(CallExpr *CE);

clang/include/clang/3C/ConstraintVariables.h

@@ -83,7 +83,7 @@ class ConstraintVariable {
   // The 'emitName' parameter is true when the generated string should include
   // the name of the variable, false for just the type.
   // The 'forIType' parameter is true when the generated string is expected
-  // to be used inside an itype
+  // to be used inside an itype.
   virtual std::string mkString(const EnvironmentMap &E, bool EmitName = true,
                                bool ForItype = false, bool EmitPointee = false,
                                bool UnmaskTypedef = false) const = 0;
@@ -186,7 +186,7 @@ class FunctionVariableConstraint;
 // We need to store the level inside the type AST at which the first
 // typedef occurs. This allows us to stop rewriting once we hit the
 // first typedef. (All subsequent typedefs will not be rewritten, as
-// rewriting will stop)
+// rewriting will stop.)
 struct InternalTypedefInfo {
   bool HasTypedef;
   int TypedefLevel;
@@ -489,7 +489,7 @@ class FunctionVariableConstraint : public ConstraintVariable {
   // Flag to indicate whether this is a function pointer or not.
   bool IsFunctionPtr;
 
-  // Count of type parameters from `_Itype_for_any(...)`
+  // Count of type parameters from `_Itype_for_any(...)`.
   int TypeParams;
 
   void equateFVConstraintVars(ConstraintVariable *CV, ProgramInfo &Info) const;

clang/include/clang/3C/ProgramInfo.h

@@ -125,12 +125,12 @@ class ProgramInfo : public ProgramVariableAdder {
   // analysis from compilation unit to compilation unit.
   VariableMap Variables;
 
-  // Map storing constraint information for typedefed types
-  // The set contains all the constraint variables that also use this tyepdef
-  // TODO this could be replaced w/ a signle CVar
+  // Map storing constraint information for typedefed types,
+  // The set contains all the constraint variables that also use this typedef.
+  // TODO this could be replaced w/ a single CVar.
   // The bool informs the rewriter whether or not this typedef should be
   // rewritten. It will be false for typedefs we don't support rewritting,
-  // such as typedefs that are pointers to anonymous structs
+  // such as typedefs that are pointers to anonymous structs.
   std::map<PersistentSourceLoc, std::pair<CVarSet, bool>> TypedefVars;
 
   // Map with the same purpose as the Variables map, this stores constraint
@@ -153,7 +153,7 @@ class ProgramInfo : public ProgramVariableAdder {
   // has been seen before.
   std::map<std::string, bool> ExternGVars;
 
-  // Maps for global/static functions, global variables
+  // Maps for global/static functions, global variables.
   ExternalFunctionMapType ExternalFunctionFVCons;
   StaticFunctionMapType StaticFunctionFVCons;
   std::map<std::string, std::set<PVConstraint *>> GlobalVariableSymbols;

clang/include/clang/3C/RewriteUtils.h

@@ -32,7 +32,7 @@ class DeclReplacement {
     return getDecl()->getSourceRange();
   }
 
-  // Discriminator for LLVM-style RTTI (dyn_cast<> et al.)
+  // Discriminator for LLVM-style RTTI (dyn_cast<> et al.).
   enum DRKind {
     DRK_VarDecl,
     DRK_ParmVarDecl,

clang/include/clang/3C/Utils.h

@@ -199,7 +199,7 @@ bool evaluateToInt(clang::Expr *E, const clang::ASTContext &C, int &Result);
 bool isZeroBoundsExpr(clang::BoundsExpr *BE, const clang::ASTContext &C);
 
 // Find the range in the source code for the base type of a type location.
-// The base type is the type after removing all
+// The base type is the type after removing all.
 clang::TypeLoc getBaseTypeLoc(clang::TypeLoc T);
 
 // Ignore all CheckedC temporary and clang implicit expression on E. This

clang/lib/3C/3C.cpp

@@ -494,7 +494,7 @@ bool _3CInterface::writeAllConvertedFilesToDisk() {
 
   ClangTool &Tool = getGlobalClangTool();
 
-  // Rewrite the input files
+  // Rewrite the input files.
   std::unique_ptr<ToolAction> RewriteTool =
       newFrontendActionFactoryA<RewriteAction<RewriteConsumer, ProgramInfo>>(
           GlobalProgramInfo, VerifyDiagnosticOutput);

clang/lib/3C/AVarBoundsInfo.cpp

@@ -811,10 +811,10 @@ bool AvarBoundsInference::predictBounds(BoundsKey K,
   }
 
   if (!InferredNBnds.empty()) {
-    // All the possible inferred bounds for K
+    // All the possible inferred bounds for K.
     InferredKBnds.clear();
     std::set<BoundsKey> TmpBKeys, AllKeys;
-    // TODO: Figure out if there is a discrepency and try to implement
+    // TODO: Figure out if there is a discrepancy and try to implement
     // root-cause analysis.
 
     // Find intersection of all bounds from neighbours.
@@ -1056,7 +1056,7 @@ void AVarBoundsInfo::computerArrPointers(ProgramInfo *PI,
       if (hasArray(PI->getVarMap().at(PSL), CS)) {
         ArrPointers.insert(Bkey);
       }
-      // Does this array belongs to a valid program variable?
+      // Does this array belong to a valid program variable?
       if (isInSrcArray(PI->getVarMap().at(PSL), CS)) {
         InProgramArrPtrBoundsKeys.insert(Bkey);
       }
@@ -1081,7 +1081,7 @@ void AVarBoundsInfo::computerArrPointers(ProgramInfo *PI,
       if (hasArray(FV->getExternalParam(ParmNum), CS)) {
         ArrPointers.insert(Bkey);
       }
-      // Does this array belongs to a valid program variable?
+      // Does this array belong to a valid program variable?
       if (isInSrcArray(FV->getExternalParam(ParmNum), CS)) {
         InProgramArrPtrBoundsKeys.insert(Bkey);
       }

clang/lib/3C/CastPlacement.cpp

@@ -46,7 +46,7 @@ bool CastPlacementVisitor::VisitCallExpr(CallExpr *CE) {
   if (Info.hasTypeParamBindings(CE, Context))
     TypeVars = Info.getTypeParamBindings(CE, Context);
 
-  // Cast on arguments
+  // Cast on arguments.
   unsigned PIdx = 0;
   for (const auto &A : CE->arguments()) {
     if (PIdx < FV->numParams()) {