Merge pull request #2636 from practicalswift/fix-plural-issues

[gardening] Fix plural issues.

Author: gribozavr

docs/SIL.rst

@@ -109,7 +109,7 @@ IR.
   functions. Then it rewrites all specialized usages of the generic
   to a direct call of the appropriate specialized function.
 - **Witness and VTable Devirtualization** for a given type looks up
-  the associated method from a class's vtable or a types witness table
+  the associated method from a class's vtable or a type witness table
   and replaces the indirect virtual call with a call to the mapped
   function.
 - **Performance Inlining**

docs/proposals/WholeModuleOptimization.rst

@@ -102,7 +102,7 @@ function individually based on those results.
 
 **TBD: Do we really need SCC transforms at all, or is it sufficient to
 simply have function transforms that are always passed an SCC, and
-have them ask for the results of an analyses for the entire SCC and
+have them ask for the results of an analysis for the entire SCC and
 then iterate over all functions in the SCC?**
 
 In some cases we have transforms that generate new work in a top-down

docs/proposals/archive/ProgramStructureAndCompilationModel.rst

@@ -255,10 +255,10 @@ components (similar to clang -F or -L) etc. We'll also support a "clean" command
 that nukes buildcache/ and products/.
 
 The BuildCache directory holds object files, dependence information and other
-stuff needed for incremental [re]builds within the component The generated
-manifest file is used both the compiler when a clients lib/app import mylib (it
+stuff needed for incremental [re]builds within the component. The generated
+manifest file is used by the compiler when a client lib/app import mylib (it
 contains type information for all the stuff exported from mylib) but also at
-runtime by the runtime library (e.g.  for reflection). It needs to be a
+runtime by the runtime library (e.g. for reflection). It needs to be a
 fast-to-read but extensible format.
 
 What the build system does, how it works

include/swift/Frontend/FrontendOptions.h

@@ -112,7 +112,7 @@ class FrontendOptions {
   /// The path to which we should output a Swift reference dependencies file.
   std::string ReferenceDependenciesFilePath;
 
-  /// The path to which we should output a fixits as source edits.
+  /// The path to which we should output fixits as source edits.
   std::string FixitsOutputPath;
 
   /// Arguments which should be passed in immediate mode.

include/swift/SIL/SILArgument.h

@@ -77,7 +77,7 @@ SILFunctionType::getSILArgumentConvention(unsigned index) const {
 
 enum class InoutAliasingAssumption {
   /// Assume that an inout indirect parameter may alias other objects.
-  /// This is the safe assumption an optimizations should make if it may break
+  /// This is the safe assumption an optimization should make if it may break
   /// memory safety in case the inout aliasing rule is violation.
   Aliasing,
 

lib/LLVMPasses/LLVMMergeFunctions.cpp

@@ -205,7 +205,7 @@ class FunctionComparator {
   /// Let's explain the order. Float numbers will be less than integers, just
   /// because of cmpType terms: FloatTyID < IntegerTyID.
   /// Floats (with same fltSemantics) are sorted according to their value.
-  /// Then you can see integers, and they are, like a floats,
+  /// Then you can see integers, and they are, like floats, which
   /// could be easy sorted among each others.
   /// The structures. Structures are grouped at the tail, again because of their
   /// TypeID: StructTyID > IntegerTyID > FloatTyID.

lib/SILOptimizer/Analysis/ARCAnalysis.cpp

@@ -722,8 +722,8 @@ void
 ConsumedArgToEpilogueReleaseMatcher::
 processMatchingReleases() {
   llvm::DenseSet<SILArgument *> ArgToRemove;
-  // If we can not find a releases for all parts with reference semantics
-  // that means we did not find all release for the base.
+  // If we can not find a release for all parts with reference semantics
+  // that means we did not find all releases for the base.
   for (auto Arg : ArgInstMap) {
     // If an argument has a single release and it is rc-identical to the
     // SILArgument. Then we do not need to use projection to check for whether

lib/SILOptimizer/Transforms/DeadCodeElimination.cpp

@@ -112,7 +112,7 @@ class DCE : public SILFunctionTransform {
     auto* DA = PM->getAnalysis<PostDominanceAnalysis>();
     PDT = DA->get(F);
 
-    // If we have a functions that consists of nothing but a
+    // If we have a function that consists of nothing but a
     // structurally infinite loop like:
     //   while true {}
     // we'll have an empty post dominator tree.

stdlib/public/core/Unmanaged.swift

@@ -139,7 +139,7 @@ public struct Unmanaged<Instance : AnyObject> {
   ///    }
   ///  }
   ///
-  /// The last rule applies transitively through a chains of stored references
+  /// The last rule applies transitively through a chain of stored references
   /// and nested structs.
   ///
   /// Examples:

utils/cmpcodesize/cmpcodesize/main.py

@@ -66,7 +66,7 @@ def main():
         cmpcodesize test.o newversion.o
 
 4) Two lists of files, separated by '--':
-    Compares a set a files.
+    Compares a set of files.
     Example:
         cmpcodesize olddir/*.o -- newdir/*.o