Fix typos.

Author: practicalswift

Diff for: include/swift/AST/ArchetypeBuilder.h

@@ -176,7 +176,7 @@ class ArchetypeBuilder {
   void visitPotentialArchetypes(F f);
 
 public:
-  /// Construct a new archtype builder.
+  /// Construct a new archetype builder.
   ///
   /// \param mod The module in which the builder will create archetypes.
   ///

Diff for: include/swift/AST/GenericSignature.h

@@ -101,7 +101,7 @@ class GenericSignature : public llvm::FoldingSetNode {
              NumGenericParams };
   }
 
-  /// Retrieve a mutable verison of the requirements.
+  /// Retrieve a mutable version of the requirements.
   MutableArrayRef<Requirement> getRequirementsBuffer() {
     void *genericParams = getGenericParamsBuffer().end();
     return { reinterpret_cast<Requirement *>(genericParams),

Diff for: include/swift/Runtime/Metadata.h

@@ -93,7 +93,7 @@ struct OpaqueValue;
 /// A buffer can be in one of three states:
 ///  - An unallocated buffer has a completely unspecified state.
 ///  - An allocated buffer has been initialized so that it
-///    owns unintialized value storage for the stored type.
+///    owns uninitialized value storage for the stored type.
 ///  - An initialized buffer is an allocated buffer whose value
 ///    storage has been initialized.
 struct ValueBuffer {
@@ -247,9 +247,9 @@ typedef void destroyBuffer(ValueBuffer *buffer, const Metadata *self);
 /// Given an unallocated buffer, initialize it as a copy of the
 /// object in the source buffer.  This can be decomposed as:
 ///
-///   self->initalizeBufferWithCopy(dest, self->projectBuffer(src), self)
+///   self->initializeBufferWithCopy(dest, self->projectBuffer(src), self)
 ///
-/// This operation does not need to be safe aginst 'dest' and 'src' aliasing.
+/// This operation does not need to be safe against 'dest' and 'src' aliasing.
 /// 
 /// Preconditions:
 ///   'dest' is an unallocated buffer
@@ -305,7 +305,7 @@ typedef OpaqueValue *initializeBufferWithCopy(ValueBuffer *dest,
 /// Given an uninitialized object and an initialized object, copy
 /// the value.
 ///
-/// This operation does not need to be safe aginst 'dest' and 'src' aliasing.
+/// This operation does not need to be safe against 'dest' and 'src' aliasing.
 /// 
 /// Returns the dest object.
 ///
@@ -322,7 +322,7 @@ typedef OpaqueValue *initializeWithCopy(OpaqueValue *dest,
 /// Given two initialized objects, copy the value from one to the
 /// other.
 ///
-/// This operation must be safe aginst 'dest' and 'src' aliasing.
+/// This operation must be safe against 'dest' and 'src' aliasing.
 /// 
 /// Returns the dest object.
 ///
@@ -337,7 +337,7 @@ typedef OpaqueValue *assignWithCopy(OpaqueValue *dest,
 /// the value from the object to the buffer, leaving the source object
 /// uninitialized.
 ///
-/// This operation does not need to be safe aginst 'dest' and 'src' aliasing.
+/// This operation does not need to be safe against 'dest' and 'src' aliasing.
 /// 
 /// Returns the dest object.
 ///
@@ -359,7 +359,7 @@ typedef OpaqueValue *initializeBufferWithTake(ValueBuffer *dest,
 /// can simply be a pointer-aligned memcpy of sizeof(ValueBuffer)
 /// bytes.
 ///
-/// This operation does not need to be safe aginst 'dest' and 'src' aliasing.
+/// This operation does not need to be safe against 'dest' and 'src' aliasing.
 /// 
 /// Returns the dest object.
 ///
@@ -377,7 +377,7 @@ typedef OpaqueValue *initializeWithTake(OpaqueValue *dest,
 /// the value from one to the other, leaving the source object
 /// uninitialized.
 ///
-/// This operation does not need to be safe aginst 'dest' and 'src' aliasing.
+/// This operation does not need to be safe against 'dest' and 'src' aliasing.
 /// Therefore this can be decomposed as:
 ///
 ///   self->destroy(dest, self);
@@ -411,10 +411,10 @@ typedef OpaqueValue *allocateBuffer(ValueBuffer *buffer,
 /// value from one buffer to the other, leaving the source buffer
 /// unallocated.
 ///
-/// This operation does not need to be safe aginst 'dest' and 'src' aliasing.
+/// This operation does not need to be safe against 'dest' and 'src' aliasing.
 /// Therefore this can be decomposed as:
 ///
-///   self->initalizeBufferWithTake(dest, self->projectBuffer(src), self)
+///   self->initializeBufferWithTake(dest, self->projectBuffer(src), self)
 ///   self->deallocateBuffer(src, self)
 ///
 /// However, it may be more efficient because values stored out-of-line
@@ -447,7 +447,7 @@ typedef void destroyArray(OpaqueValue *array, size_t n,
 /// Given an uninitialized array and an initialized array, copy
 /// the value.
 ///
-/// This operation does not need to be safe aginst 'dest' and 'src' aliasing.
+/// This operation does not need to be safe against 'dest' and 'src' aliasing.
 /// 
 /// Returns the dest object.
 ///

Diff for: include/swift/SILOptimizer/Analysis/EscapeAnalysis.h

@@ -736,7 +736,7 @@ class EscapeAnalysis : public BottomUpIPAnalysis {
   bool canEscapeTo(SILValue V, RefCountingInst *RI);
 
   /// Returns true if the value \p V can escape to any other pointer \p To.
-  /// This means that either \p To is the same as \p V or containes a reference
+  /// This means that either \p To is the same as \p V or contains a reference
   /// to \p V.
   bool canEscapeToValue(SILValue V, SILValue To);
 

Diff for: lib/AST/Mangle.cpp

@@ -446,7 +446,7 @@ static void bindAllGenericParameters(Mangler &mangler,
 }
 
 void Mangler::mangleTypeForDebugger(Type Ty, const DeclContext *DC) {
-  assert(DWARFMangling && "DWARFMangling expected whn mangling for debugger");
+  assert(DWARFMangling && "DWARFMangling expected when mangling for debugger");
 
   // Polymorphic function types carry their own generic parameters and
   // manglePolymorphicType will bind them.

Diff for: lib/ClangImporter/ImportDecl.cpp

@@ -747,10 +747,10 @@ makeUnionFieldAccessors(ClangImporter::Implementation &Impl,
       C, C.getIdentifier("initialize")));
     auto initializeFnRef
       = new (C) DeclRefExpr(initializeFn, SourceLoc(), /*implicit*/ true);
-    auto initalizeArgs = TupleExpr::createImplicit(C,
+    auto initializeArgs = TupleExpr::createImplicit(C,
                                                    { newValueRef, selfPointer },
                                                    {});
-    auto initialize = new (C) CallExpr(initializeFnRef, initalizeArgs,
+    auto initialize = new (C) CallExpr(initializeFnRef, initializeArgs,
                                        /*implicit*/ true);
     auto body = BraceStmt::create(C, SourceLoc(), { initialize }, SourceLoc(),
                                   /*implicit*/ true);

Diff for: lib/IDE/Utils.cpp

@@ -189,7 +189,7 @@ SourceCompleteResult ide::isSourceInputComplete(StringRef Text) {
 }
 
 // Adjust the cc1 triple string we got from clang, to make sure it will be
-// accepted when it goes throught the swift clang importer.
+// accepted when it goes through the swift clang importer.
 static std::string adjustClangTriple(StringRef TripleStr) {
   std::string Result;
   llvm::raw_string_ostream OS(Result);

Diff for: lib/IRGen/GenFunc.cpp

@@ -1830,7 +1830,7 @@ if (Builtin.ID == BuiltinValueKind::id) { \
   return; \
 }
   // FIXME: We could generate the code to dynamically report the overflow if the
-  // thrid argument is true. Now, we just ignore it.
+  // third argument is true. Now, we just ignore it.
 
 #define BUILTIN_BINARY_PREDICATE(id, name, attrs, overload) \
   if (Builtin.ID == BuiltinValueKind::id) \

Diff for: lib/IRGen/NonFixedTypeInfo.h

@@ -15,7 +15,7 @@
 //  statically.
 //
 //  These classes are useful only for creating TypeInfo
-//  implementations; unlike the similiarly-named FixedTypeInfo, they
+//  implementations; unlike the similarly-named FixedTypeInfo, they
 //  do not provide a supplemental API.
 //
 //===----------------------------------------------------------------------===//

Diff for: lib/IRGen/ValueWitness.h

@@ -83,7 +83,7 @@ enum class ValueWitness : unsigned {
   ///   T *(*initializeBufferWithCopyOfBuffer)(B *dest, B *src, M *self);
   /// Given an invalid buffer, initialize it as a copy of the
   /// object in the source buffer.  This can be decomposed as:
-  ///   initalizeBufferWithCopy(dest, self->projectBuffer(src), self)
+  ///   initializeBufferWithCopy(dest, self->projectBuffer(src), self)
   InitializeBufferWithCopyOfBuffer,
 
   ///   T *(*projectBuffer)(B *buffer, M *self);
@@ -156,7 +156,7 @@ enum class ValueWitness : unsigned {
   ///   T *(*initializeBufferWithTakeOfBuffer)(B *dest, B *src, M *self);
   /// Given an invalid buffer, initialize it by taking the value out of
   /// the source buffer.  This can be (inefficiently) decomposed as:
-  ///   initalizeBufferWithTake(dest, self->projectBuffer(src), self)
+  ///   initializeBufferWithTake(dest, self->projectBuffer(src), self)
   ///   deallocateBuffer(src, self)
   InitializeBufferWithTakeOfBuffer,
 

Diff for: lib/LLVMPasses/LLVMARCOpts.cpp

@@ -856,7 +856,7 @@ static bool performLocalRetainUnownedOpt(CallInst *Retain, BasicBlock &BB,
 }
 
 /// Removes redundant check_unowned calls if they check the same reference and
-/// there is no instruction inbetween which could decrement the reference count.
+/// there is no instruction in between which could decrement the reference count.
 static void performRedundantCheckUnownedRemoval(BasicBlock &BB) {
   DenseSet<Value *> checkedValues;
   for (BasicBlock::iterator BBI = BB.begin(), E = BB.end(); BBI != E; ) {

Diff for: lib/Parse/ParseStmt.cpp

@@ -700,7 +700,7 @@ ParserResult<Stmt> Parser::parseStmtReturn(SourceLoc tryLoc) {
     ParserResult<Expr> Result = parseExpr(diag::expected_expr_return);
     if (Result.isNull()) {
       // Create an ErrorExpr to tell the type checker that this return
-      // statement had an expression argument in the source.  This supresses
+      // statement had an expression argument in the source.  This suppresses
       // the error about missing return value in a non-void function.
       Result = makeParserErrorResult(new (Context) ErrorExpr(ExprLoc));
     }

Diff for: lib/SILOptimizer/IPO/GlobalOpt.cpp

@@ -549,7 +549,7 @@ static bool isAssignedOnlyOnceInInitializer(SILGlobalVariable *SILG) {
   return false;
 }
 
-/// Replace load sequence which may contian
+/// Replace load sequence which may contain
 /// a chain of struct_element_addr followed by a load.
 /// The sequence is traversed starting from the load
 /// instruction.

Diff for: lib/SILOptimizer/Mandatory/ConstantPropagation.cpp

@@ -481,7 +481,7 @@ constantFoldAndCheckIntegerConversions(BuiltinInst *BI,
     // 2048. Is there a better way to identify conversions from literals?
     bool Literal = (SrcBitWidth == 2048);
 
-    // FIXME: This will prevent hard error in cases the error is comming
+    // FIXME: This will prevent hard error in cases the error is coming
     // from ObjC interoperability code. Currently, we treat NSUInteger as
     // Int.
     if (Loc.getSourceLoc().isInvalid()) {

Diff for: lib/SILOptimizer/SILCombiner/SILCombinerMiscVisitors.cpp

@@ -666,7 +666,7 @@ SILCombiner::visitInjectEnumAddrInst(InjectEnumAddrInst *IEAI) {
           if (SI->getDest() == IEAI->getOperand())
             return nullptr;
         }
-        // Allow all instructions inbetween, which don't have any dependency to
+        // Allow all instructions in between, which don't have any dependency to
         // the store.
         if (AA->mayWriteToMemory(&*II, IEAI->getOperand()))
           return nullptr;
@@ -705,7 +705,7 @@ SILCombiner::visitInjectEnumAddrInst(InjectEnumAddrInst *IEAI) {
           if (SI->getDest() == IEAI->getOperand())
             return nullptr;
         }
-        // Allow all instructions inbetween, which don't have any dependency to
+        // Allow all instructions in between, which don't have any dependency to
         // the store.
         if (AA->mayWriteToMemory(&*II, IEAI->getOperand()))
           return nullptr;

Diff for: lib/SILOptimizer/Transforms/SILCodeMotion.cpp

@@ -898,7 +898,7 @@ static bool isRetainAvailableInSomeButNotAllPredecessors(
     // Check that there is no decrement or check from the increment to the end
     // of the basic block. After we have hoisted the first release  this release
     // would prevent further hoisting. Instead we check that no decrement or
-    // check occurs upto this hoisted release.
+    // check occurs up to this hoisted release.
     auto End = CheckUpToInstruction[Pred];
     auto EndIt = SILBasicBlock::iterator(End ? *End : Pred->getTerminator());
     if (Retain == Pred->rend() || valueHasARCDecrementOrCheckInInstructionRange(

Diff for: lib/SILOptimizer/Utils/Local.cpp

@@ -2299,7 +2299,7 @@ swift::analyzeStaticInitializer(SILValue V,
   return false;
 }
 
-/// Replace load sequence which may contian
+/// Replace load sequence which may contain
 /// a chain of struct_element_addr followed by a load.
 /// The sequence is travered inside out, i.e.
 /// starting with the innermost struct_element_addr

Diff for: lib/SILOptimizer/Utils/SILSSAUpdater.cpp

@@ -175,7 +175,7 @@ SILValue SILSSAUpdater::GetValueInMiddleOfBlock(SILBasicBlock *BB) {
   bool FirstPred = true;
 
   // SSAupdater can modify TerminatorInst and therefore invalidate the
-  // predecessor iterator. Find all the predecesors before the SSA update.
+  // predecessor iterator. Find all the predecessors before the SSA update.
   SmallVector<SILBasicBlock *, 4> Preds;
   for (auto *PredBB: BB->getPreds()) {
     Preds.push_back(PredBB);

Diff for: stdlib/public/core/StringBuffer.swift

@@ -186,7 +186,7 @@ public struct _StringBuffer {
   /// Attempt to claim unused capacity in the buffer.
   ///
   /// Operation succeeds if there is sufficient capacity, and either:
-  /// - the buffer is uniquely-refereced, or
+  /// - the buffer is uniquely-referenced, or
   /// - `oldUsedEnd` points to the end of the currently used capacity.
   ///
   /// - parameter subRange: Range of the substring that the caller tries

Diff for: stdlib/public/core/UnavailableStringAPIs.swift.gyb

@@ -38,7 +38,7 @@ stringSubscriptComment = """
   ///   Use this API when you are performing low-level manipulation
   ///   of character data.
   ///
-  /// - `String.chracters` is a collection of extended grapheme
+  /// - `String.characters` is a collection of extended grapheme
   ///   clusters, which are an approximation of user-perceived
   ///   characters.
   ///
@@ -98,7 +98,7 @@ ${stringSubscriptComment}
   ///   Unicode scalars in the string.  Use this API when you are
   ///   performing low-level manipulation of character data.
   ///
-  /// - `String.chracters.count` property returns the number of
+  /// - `String.characters.count` property returns the number of
   ///   extended grapheme clusters.  Use this API to count the
   ///   number of user-perceived characters in the string.
   @available(

Diff for: stdlib/public/core/UnsafePointer.swift.gyb

@@ -164,7 +164,7 @@ ${comment}
   ///
   /// - Precondition: The memory is not initialized.
   ///
-  /// - Postcondition: The memory is initalized; the value should eventually
+  /// - Postcondition: The memory is initialized; the value should eventually
   ///   be destroyed or moved from to avoid leaks.
   public func initialize(newvalue: Memory) {
     Builtin.initialize(newvalue, _rawValue)

Diff for: test/Interpreter/SDK/missing_imports_repl.swift

@@ -15,10 +15,10 @@ import MapKit
 let x = MKMapRectMake(0.0, 1.0, 2.0, 3.0)
 // CHECK-NEXT: x : MKMapRect
 
-import Nonexistant_Module_Name
-// CHECK-ERROR: error: no such module 'Nonexistant_Module_Name'
+import Nonexistent_Module_Name
+// CHECK-ERROR: error: no such module 'Nonexistent_Module_Name'
 
-import SpriteKit.Nonexistant_Submodule
+import SpriteKit.Nonexistent_Submodule
 // CHECK-ERROR: error: no such module
 
 SKScene()

Diff for: test/SILOptimizer/definite_init_diagnostics.swift

@@ -145,7 +145,7 @@ func test4() {
   markUsed(t3.2)
 
 
-  // Partially set, wholey read.
+  // Partially set, wholly read.
   var t4 : (Int, Int, Int)   // expected-note 1 {{variable defined here}}
   t4.0 = 1; t4.2 = 42
   _ = t4            // expected-error {{variable 't4.1' used before being initialized}}

Diff for: test/SILOptimizer/globalarcopts_rcidentityanalysis.sil

@@ -287,10 +287,10 @@ bb6(%4 : $FakeOptional<Builtin.NativeObject>):
   return %9999 : $()
 }
 
-// CHECK-LABEL: sil @silargument_dont_strip_over_relevent_loop : $@convention(thin) (FakeOptional<Builtin.NativeObject>) -> () {
+// CHECK-LABEL: sil @silargument_dont_strip_over_relevant_loop : $@convention(thin) (FakeOptional<Builtin.NativeObject>) -> () {
 // CHECK: retain_value
 // CHECK: release_value
-sil @silargument_dont_strip_over_relevent_loop : $@convention(thin) (FakeOptional<Builtin.NativeObject>) -> () {
+sil @silargument_dont_strip_over_relevant_loop : $@convention(thin) (FakeOptional<Builtin.NativeObject>) -> () {
 bb0(%0 : $FakeOptional<Builtin.NativeObject>):
   switch_enum %0 : $FakeOptional<Builtin.NativeObject>, case #FakeOptional.Some!enumelt.1: bb1, case #FakeOptional.None!enumelt: bb2
 

Diff for: test/SILOptimizer/linker.swift

@@ -6,7 +6,7 @@
 // CHECK: sil public_external [fragile] @_TFs11doSomethingFT_T_ : $@convention(thin) () -> () {
 doSomething()
 
-// Make sure we are not linking doSomethign2 because it is marked with 'noimport'
+// Make sure we are not linking doSomething2 because it is marked with 'noimport'
 
 // CHECK: sil [_semantics "stdlib_binary_only"] @_TFs12doSomething2FT_T_ : $@convention(thin) () -> ()
 // CHECK-NOT: return

Diff for: test/decl/import/import.swift

@@ -27,7 +27,7 @@ func f1(a: Swift.Int) -> Swift.Void { print(a) }
 import func Swift.print
 
 // rdar://14418336
-#import something_nonexistant // expected-error {{expected expression}} expected-error {{no such module 'something_nonexistant'}}
+#import something_nonexistent // expected-error {{expected expression}} expected-error {{no such module 'something_nonexistent'}}
 
 // Import specific decls
 import typealias Swift.Int

Diff for: tools/SourceKit/tools/sourcekitd/include/sourcekitd/sourcekitd.h

@@ -122,7 +122,7 @@ typedef void(^sourcekitd_interrupted_connection_handler_t)(void);
 
 /**
  * \brief Sets the handler which should be called whenever the connection to
- * SourceKit is interupted.
+ * SourceKit is interrupted.
  *
  * The handler should reestablish any necessary state, such as re-opening any
  * documents which were open before the connection was interrupted.

Diff for: utils/build-script-impl

@@ -51,7 +51,7 @@ LLVM_TARGETS_TO_BUILD="X86;ARM;AArch64"
 # components).
 #
 # Each variable name is re-exported into this script in uppercase, where dashes
-# are substituded by underscores. For example, `swift-install-components` is
+# are substituted by underscores. For example, `swift-install-components` is
 # referred to as `SWIFT_INSTALL_COMPONENTS` in the remainder of this script.
 KNOWN_SETTINGS=(
     # name                      default          description