Spelling

CODE_OWNERS.TXT

@@ -37,7 +37,7 @@ E: [email protected]
 G: jckarter
 D: SIL, SILGen, everything in Swift not covered by someone else
 
-N: Argyrios Kyrzidis
+N: Argyrios Kyrtzidis
 E: [email protected]
 G: akyrtzi
 D: IDE, Index, SourceKit, swift-ide-test

docs/proposals/Accessors.rst

@@ -430,7 +430,7 @@ and crashes are acceptable in their wake.)
 
 However, addressors are not atomic in this way: they return an address
 to the caller, which may then interleave arbitrary code before
-completing the operation.  This can present the opportunity for
+completion the operation.  This can present the opportunity for
 corruption if the interleaved code modifies the original value.
 Consider the following code::
 

include/swift/ABI/Metadata.h

@@ -877,7 +877,7 @@ struct TargetClassMetadata : public TargetAnyClassMetadataVariant {
   //
   // Using Objective-C runtime, KVO can modify object behavior without needing
   // to modify the object's code. This is done by dynamically creating an
-  // artificial subclass of the the object's type.
+  // artificial subclass of the object's type.
   //
   // The isa pointer of the observed object is swapped out to point to
   // the artificial subclass, which has the following properties:

include/swift/ABI/TaskOptions.h

@@ -78,7 +78,7 @@ class TaskGroupTaskOptionRecord : public TaskOptionRecord {
 
 /// Task option to specify on what executor the task should be executed.
 ///
-/// Not passing this option implies that that a "best guess" or good default
+/// Not passing this option implies that a "best guess" or good default
 /// executor should be used instead, most often this may mean the global
 /// concurrent executor, or the enclosing actor's executor.
 class ExecutorTaskOptionRecord : public TaskOptionRecord {

include/swift/Concurrency/Actor.h

@@ -39,7 +39,7 @@ enum : uint32_t {
   // It is possible for an actor to be in Running and yet completely released
   // by clients. However, the actor needs to be kept alive until it is done
   // executing the task that is running on it and gives it up. It is only
-  // after that that we can safely deallocate it.
+  // after that we can safely deallocate it.
   ActorStateMask = 0x7,
 
   /// The actor is not currently scheduled.  Completely redundant

include/swift/Demangling/Demangler.h

@@ -623,7 +623,7 @@ class Demangler : public NodeFactory {
 
 /// A demangler which uses stack space for its initial memory.
 ///
-/// The \p Size paramter specifies the size of the stack space.
+/// The \p Size parameter specifies the size of the stack space.
 template <size_t Size> class StackAllocatedDemangler : public Demangler {
   char StackSpace[Size];
 

include/swift/Reflection/TypeRefBuilder.h

@@ -1033,13 +1033,13 @@ class TypeRefBuilder {
       auto parentOffsetAddress = detail::applyRelativeOffset(
           (const char *)contextDescriptorAddress,
           (int32_t)contextDescriptor->getParentOffset());
-      auto parentOfsetBytes = OpaqueByteReader(
+      auto parentOffsetBytes = OpaqueByteReader(
           remote::RemoteAddress(parentOffsetAddress), sizeof(uint32_t));
-      if (!parentOfsetBytes.get()) {
+      if (!parentOffsetBytes.get()) {
         Error = "Failed to parent offset in a type descriptor.";
         return llvm::None;
       }
-      auto parentFieldOffset = (const int32_t *)parentOfsetBytes.get();
+      auto parentFieldOffset = (const int32_t *)parentOffsetBytes.get();
       auto parentTargetAddress = detail::applyRelativeOffset(
           (const char *)parentOffsetAddress, *parentFieldOffset);
 
@@ -1128,13 +1128,13 @@ class TypeRefBuilder {
       auto typeNameOffsetAddress =
           detail::applyRelativeOffset((const char *)typeDescriptorAddress,
                                       (int32_t)typeDescriptor->getNameOffset());
-      auto typeNameOfsetBytes = OpaqueByteReader(
+      auto typeNameOffsetBytes = OpaqueByteReader(
           remote::RemoteAddress(typeNameOffsetAddress), sizeof(uint32_t));
-      if (!typeNameOfsetBytes.get()) {
+      if (!typeNameOffsetBytes.get()) {
         Error = "Failed to read type name offset in a type descriptor.";
         return llvm::None;
       }
-      auto typeNameOffset = (const uint32_t *)typeNameOfsetBytes.get();
+      auto typeNameOffset = (const uint32_t *)typeNameOffsetBytes.get();
       auto typeNameAddress = detail::applyRelativeOffset(
           (const char *)typeNameOffsetAddress, (int32_t)*typeNameOffset);
       std::string typeName;
@@ -1149,15 +1149,15 @@ class TypeRefBuilder {
       auto parentNameOffsetAddress = detail::applyRelativeOffset(
           (const char *)moduleDescriptorAddress,
           (int32_t)moduleDescriptor->getNameOffset());
-      auto parentNameOfsetBytes = OpaqueByteReader(
+      auto parentNameOffsetBytes = OpaqueByteReader(
           remote::RemoteAddress(parentNameOffsetAddress), sizeof(uint32_t));
-      if (!parentNameOfsetBytes.get()) {
+      if (!parentNameOffsetBytes.get()) {
         Error = "Failed to read parent name offset in a module descriptor.";
         return llvm::None;
       }
-      auto parentNameOfset = (const uint32_t *)parentNameOfsetBytes.get();
+      auto parentNameOffset = (const uint32_t *)parentNameOffsetBytes.get();
       auto parentNameAddress = detail::applyRelativeOffset(
-          (const char *)parentNameOffsetAddress, (int32_t)*parentNameOfset);
+          (const char *)parentNameOffsetAddress, (int32_t)*parentNameOffset);
       std::string parentName;
       OpaqueStringReader(remote::RemoteAddress(parentNameAddress), parentName);
       return parentName;
@@ -1183,13 +1183,13 @@ class TypeRefBuilder {
       auto protocolNameOffsetAddress = detail::applyRelativeOffset(
           (const char *)protocolDescriptorAddress,
           (int32_t)protocolDescriptor->getNameOffset());
-      auto protocolNameOfsetBytes = OpaqueByteReader(
+      auto protocolNameOffsetBytes = OpaqueByteReader(
           remote::RemoteAddress(protocolNameOffsetAddress), sizeof(uint32_t));
-      if (!protocolNameOfsetBytes.get()) {
+      if (!protocolNameOffsetBytes.get()) {
         Error = "Failed to read type name offset in a protocol descriptor.";
         return llvm::None;
       }
-      auto protocolNameOffset = (const uint32_t *)protocolNameOfsetBytes.get();
+      auto protocolNameOffset = (const uint32_t *)protocolNameOffsetBytes.get();
 
       // Using the offset above, compute the address of the name field itsel
       // and read it.

include/swift/Syntax/Syntax.h

@@ -141,7 +141,7 @@ class OwnedSyntaxRef {
 
 /// Same as \c OwnedSyntaxRef but can be null. We don't use \c
 /// Optional<OwnedSyntaxRef<SyntaxRefType>>>, because then we couldn't access
-/// the underlying \c SytnaxRefType via the \c -> operator (the use of \c ->
+/// the underlying \c SyntaxRefType via the \c -> operator (the use of \c ->
 /// would access the \c OwnedSyntaxRef<SyntaxRefType> wrapped by \c Optional and
 /// not the \c SyntaxRefType wrapped by \c OwnedSyntaxRef.
 template <typename SyntaxRefType>

lib/Demangling/NodePrinter.cpp

@@ -2551,7 +2551,7 @@ NodePointer NodePrinter::print(NodePointer Node, unsigned depth,
         if (index != 0)
           Printer << ", ";
         // Limit the number of printed generic parameters. In practice this
-        // it will never be exceeded. The limit is only imporant for malformed
+        // it will never be exceeded. The limit is only important for malformed
         // symbols where count can be really huge.
         if (index >= 128) {
           Printer << "...";

lib/Localization/LocalizationFormat.cpp

@@ -231,7 +231,7 @@ readYAML(llvm::yaml::IO &io, T &Seq, T &unknownIDs, bool, Context &Ctx) {
       io.beginMapping();
 
       // If the current diagnostic ID is available in YAML and in `.def`, add it
-      // to the diagnostics array. Otherwise, re-parse the current diagnnostic
+      // to the diagnostics array. Otherwise, re-parse the current diagnostic
       // id as a string and store it in `unknownIDs` array.
       if (auto id = LocalizationInput::readID(io)) {
         // YAML file isn't guaranteed to have diagnostics in order of their

lib/Migrator/APIDiffMigratorPass.cpp

@@ -1300,7 +1300,7 @@ struct APIDiffMigratorPass : public ASTMigratorPass, public SourceEntityWalker {
     if (Params.size() <= Idx)
       return;
 
-    // Get the internal name of the changed paramter.
+    // Get the internal name of the changed parameter.
     auto VariableName = Params[Idx]->getParameterName().str();
 
     // Insert the helper function to convert the type back to raw types.
@@ -1432,7 +1432,7 @@ struct APIDiffMigratorPass : public ASTMigratorPass, public SourceEntityWalker {
     if (auto *VD = dyn_cast<VarDecl>(D)) {
       for (auto *Item: getRelatedDiffItems(VD)) {
         if (auto *CD = dyn_cast<CommonDiffItem>(Item)) {
-          // If the overriden property has been renamed, we should rename
+          // If the overridden property has been renamed, we should rename
           // this property decl as well.
           if (CD->isRename() && VD->getNameLoc().isValid()) {
             Editor.replaceToken(VD->getNameLoc(), CD->getNewName());

lib/Migrator/Diff.h

@@ -662,7 +662,7 @@ class diff_match_patch {
   static string_t diff_linesToCharsMunge(const string_t &text, std::map<LinePtr, size_t> &lineHash) {
     string_t chars;
     // Walk the text, pulling out a substring for each line.
-    // text.split('\n') would would temporarily double our memory footprint.
+    // text.split('\n') would temporarily double our memory footprint.
     // Modifying text would create many large strings to garbage collect.
     typename string_t::size_type lineLen;
     for (typename string_t::const_pointer lineStart = text.c_str(), textEnd = lineStart + text.size(); lineStart < textEnd; lineStart += lineLen + 1) {

lib/Option/SanitizerOptions.cpp

@@ -141,7 +141,7 @@ OptionSet<SanitizerKind> swift::parseSanitizerArgValues(
     }
     SanitizerKind kind = optKind.getValue();
 
-    // Support is determined by existance of the sanitizer library.
+    // Support is determined by existence of the sanitizer library.
     auto fileName = toFileName(kind);
     bool isShared = (kind != SanitizerKind::Fuzzer);
     bool sanitizerSupported = sanitizerRuntimeLibExists(fileName, isShared);

lib/PrintAsClang/DeclAndTypePrinter.cpp

@@ -289,8 +289,8 @@ class DeclAndTypePrinter::Implementation
     printDocumentationComment(CD);
 
     // This is just for testing, so we check explicitly for the attribute instead
-    // of asking if the class is weak imported. If the class has availablility,
-    // we'll print a SWIFT_AVAIALBLE() which implies __attribute__((weak_imported))
+    // of asking if the class is weak imported. If the class has availability,
+    // we'll print a SWIFT_AVAILABLE() which implies __attribute__((weak_imported))
     // already.
     if (CD->getAttrs().hasAttribute<WeakLinkedAttr>())
       os << "SWIFT_WEAK_IMPORT\n";

lib/Syntax/SyntaxSerialization.cpp.gyb

@@ -8,7 +8,7 @@
 }%
 //// Automatically Generated From SyntaxSerialization.cpp.gyb.
 //// Do Not Edit Directly!
-//===---------------------- SytnaxSerialization.cpp -----------------------===//
+//===---------------------- SyntaxSerialization.cpp -----------------------===//
 //
 // This source file is part of the Swift.org open source project
 //

lib/SyntaxParse/SyntaxTreeCreator.cpp

@@ -199,14 +199,14 @@ OpaqueSyntaxNode SyntaxTreeCreator::makeDeferredLayout(
 
 OpaqueSyntaxNode
 SyntaxTreeCreator::recordDeferredToken(OpaqueSyntaxNode deferred) {
-  // We don't diffirentiate between deferred and recorded nodes. See comment in
+  // We don't differentiate between deferred and recorded nodes. See comment in
   // makeDeferredToken.
   return deferred;
 }
 
 OpaqueSyntaxNode
 SyntaxTreeCreator::recordDeferredLayout(OpaqueSyntaxNode deferred) {
-  // We don't diffirentiate between deferred and recorded nodes. See comment in
+  // We don't differentiate between deferred and recorded nodes. See comment in
   // makeDeferredToken.
   return deferred;
 }

lib/TBDGen/TBDGen.cpp

@@ -269,7 +269,7 @@ getLinkerPlatformName(OriginallyDefinedInAttr::ActiveVersion Ver) {
   return getLinkerPlatformName((uint8_t)getLinkerPlatformId(Ver));
 }
 
-/// Find the most relevant introducing version of the decl stack we have visted
+/// Find the most relevant introducing version of the decl stack we have visited
 /// so far.
 static Optional<llvm::VersionTuple>
 getInnermostIntroVersion(ArrayRef<Decl*> DeclStack, PlatformKind Platform) {

test/CMakeLists.txt

@@ -239,8 +239,8 @@ endif()
 foreach(SDK ${SWIFT_SDKS})
   foreach(ARCH ${SWIFT_SDK_${SDK}_ARCHITECTURES})
     # macCatalyst needs to run two sets of tests: one with the normal macosx target triple
-    # and one with the the macCatalyst ios-macabi triple.  The build_flavors list will
-    # have have only the "default" flavor for all SDKs and architectures except
+    # and one with the macCatalyst ios-macabi triple.  The build_flavors list will
+    # have only the "default" flavor for all SDKs and architectures except
     # OSX when macCatalyst support is enabled.
     get_swift_test_build_flavors(build_flavors "${SDK}")
 
@@ -296,7 +296,7 @@ foreach(SDK ${SWIFT_SDKS})
             "swift-test-stdlib-${SWIFT_SDK_${SDK}_LIB_SUBDIR}")
 
         if(BUILD_FLAVOR STREQUAL "ios-like")
-          # When testing the iOS-like build flavor, use the the normal macOS
+          # When testing the iOS-like build flavor, use the normal macOS
           # swift-reflection-test-tool. That tool runs out of process so it
           # doesn't need to be build for macCatalyst.
           list(APPEND test_dependencies

test/Compatibility/tuple_arguments_4.swift

@@ -2,7 +2,7 @@
 
 // https://bugs.swift.org/browse/SR-6837
 
-// FIXME: Can't overlaod local functions so these must be top-level
+// FIXME: Can't overload local functions so these must be top-level
 func takePairOverload(_ pair: (Int, Int?)) {}
 func takePairOverload(_: () -> ()) {}
 

test/CrossImport/remark-option.swift

@@ -4,6 +4,6 @@
 // RUN: %target-typecheck-verify-swift -enable-cross-import-overlays -Rcross-import -I %t/include -I %t/lib/swift -F %t/Frameworks
 
 import DeclaringLibrary
-// FIXME: Similarly to horrible.swift, ideally we would emit this remark on DelcaringLibrary
+// FIXME: Similarly to horrible.swift, ideally we would emit this remark on DeclaringLibrary
 // decl, since the cross-import overlay actually belongs to the DeclaringLibrary. (SR-12223)
 import BystandingLibrary // expected-remark {{import of 'DeclaringLibrary' and 'BystandingLibrary' triggered a cross-import of '_OverlayLibrary'}}

test/Inputs/polymorphic_builtins.swift

@@ -55,7 +55,7 @@ extension SIMDStorage {
     @inline(never)
     get {
       // Will never be called unless `_hasVectorRepresentation == true`,
-      // in which case this implementation would be overriden in stdlib
+      // in which case this implementation would be overridden in stdlib
       fatalError("""
         Error! Called default SIMDStorage._vector impl?! A SIMDStorage class
         overrides _hasVectorRepresentation to return true, but did not provide
@@ -83,7 +83,7 @@ extension SIMDStorage {
   @inline(never)
   public init(_storage: _InnerStorage) {
     // Will never be called unless `_hasVectorRepresentation == true`, in
-    // which case this implementation would be overriden in stdlib
+    // which case this implementation would be overridden in stdlib
     fatalError("""
       Error! Called default SIMDStorage.init(_vector) impl?! A SIMDStorage class
       overrides _hasVectorRepresentation to return true, but did not provide an

test/Migrator/swift_version_change.swift

@@ -1,5 +1,5 @@
 // REQUIRES: OS=macosx
-// RUN: %empty-directory(%t) && %target-swift-frontend -typecheck -update-code -primary-file %s -emit-migrated-file-path %t/dummpy.result -swift-version 4 %api_diff_data_dir
-// RUN: %target-swift-frontend -typecheck -update-code -primary-file %s -emit-migrated-file-path %t/dummpy.result -swift-version 5 %api_diff_data_dir
+// RUN: %empty-directory(%t) && %target-swift-frontend -typecheck -update-code -primary-file %s -emit-migrated-file-path %t/dummy.result -swift-version 4 %api_diff_data_dir
+// RUN: %target-swift-frontend -typecheck -update-code -primary-file %s -emit-migrated-file-path %t/dummy.result -swift-version 5 %api_diff_data_dir
 
 func foo() {}

test/Misc/serialized-diagnostics-interpret-mode.swift

@@ -4,11 +4,11 @@
 
 // RUN: %target-swift-frontend -typecheck -serialize-diagnostics-path %t.dia %s -verify
 // RUN: %target-swift-frontend -typecheck -serialize-diagnostics-path=%t_EQ.dia %s -verify
-// RUN: not %swift_driver -serialize-diagnostics-path %t_intepret_mode.dia %s
-// RUN: not %swift_driver -serialize-diagnostics-path=%t_EQ_intepret_mode.dia %s
+// RUN: not %swift_driver -serialize-diagnostics-path %t_interpret_mode.dia %s
+// RUN: not %swift_driver -serialize-diagnostics-path=%t_EQ_interpret_mode.dia %s
 
 // RUN: diff %t.dia %t_EQ.dia
-// RUN: diff %t.dia %t_intepret_mode.dia
-// RUN: diff %t.dia %t_EQ_intepret_mode.dia
+// RUN: diff %t.dia %t_interpret_mode.dia
+// RUN: diff %t.dia %t_EQ_interpret_mode.dia
 
 var x = 1 x = 2   // expected-error {{consecutive statements on a line must be separated by ';'}} {{10-10=;}}

test/ModuleInterface/ModuleCache/SDKDependencies-disable-validation.swift

@@ -62,7 +62,7 @@
 // RUN: echo '2: PASSED'
 
 
-// 3) Baseline check: Make sure we use the the prebuilt module cache when using the SDK it was built with
+// 3) Baseline check: Make sure we use the prebuilt module cache when using the SDK it was built with
 //
 // RUN: %target-swift-frontend -typecheck -I %t/my-sdk -sdk %t/my-sdk -prebuilt-module-cache-path %t/prebuilt-cache -module-cache-path %t/MCP -emit-dependencies-path %t/dummy.d -track-system-dependencies -disable-modules-validate-system-headers %s
 //

test/ModuleInterface/ModuleCache/SDKDependencies.swift

@@ -72,7 +72,7 @@
 // RUN: echo '2: PASSED'
 
 
-// 3) Baseline check: Make sure we use the the prebuilt module cache when using the SDK it was built with
+// 3) Baseline check: Make sure we use the prebuilt module cache when using the SDK it was built with
 //
 // RUN: %target-swift-frontend -typecheck -I %t/my-sdk -sdk %t/my-sdk -prebuilt-module-cache-path %t/prebuilt-cache -module-cache-path %t/MCP -emit-dependencies-path %t/dummy.d -track-system-dependencies  -emit-loaded-module-trace-path %t/trace.json %s
 //

test/ModuleInterface/ossa-modules/sdk-test-stdlib-ossa.swift

@@ -25,7 +25,7 @@
 
 // RUN: %target-swift-frontend -typecheck -sdk '%t/SDK' -prebuilt-module-cache-path '%t/PreBuiltSDKModules' -module-cache-path %t/TempModuleCacheOther -resource-dir '' -parse-stdlib -Rmodule-interface-rebuild %S/Inputs/sdk-test-stdlib-no-ossa-referent-no-rebuild-remark.swift -verify -enable-ossa-modules
 
-// Flacky hangs: rdar://77288690
+// Flaky hangs: rdar://77288690
 // UNSUPPORTED: CPU=arm64, CPU=arm64e
 
 @_fixed_layout

test/ModuleInterface/swift_build_sdk_interfaces/compiler-crash-windows.test-sh

@@ -1,4 +1,4 @@
-# This test is intentionally specialized on windows becauase not --crash behaves
+# This test is intentionally specialized on windows because not --crash behaves
 # differently there. The exit code from the crashing frontend invocations is
 # mapped to `-21` there, which lit's `not` along does not remap correctly.
 # See rdar://59397376

test/NameLookup/Inputs/lazy_function_body_expansion_helper.swift

@@ -1,5 +1,5 @@
 // Body of closure in parameter to call of closureTaker is created lazily
-// and this test ensures that that body scope does get expanded
+// and this test ensures that body scope does get expanded
 var v = closureTaker {
   func amIFound() {}
 }

test/Profiler/converage_result_builder.swift

@@ -1,4 +1,4 @@
-// RUN: %target-swift-frontend -Xllvm -sil-full-demangle -profile-generate -profile-coverage-mapping -emit-sorted-sil -emit-sil -module-name coverage_functon_builder %s | %FileCheck %s
+// RUN: %target-swift-frontend -Xllvm -sil-full-demangle -profile-generate -profile-coverage-mapping -emit-sorted-sil -emit-sil -module-name coverage_function_builder %s | %FileCheck %s
 // RUN: %target-swift-frontend -profile-generate -profile-coverage-mapping -emit-ir %s
 
 @resultBuilder
@@ -11,15 +11,15 @@ struct Summer {
   }
 }
 
-// CHECK-LABEL: sil_coverage_map {{.*}} "$s24coverage_functon_builder5test0SiyF"
+// CHECK-LABEL: sil_coverage_map {{.*}} "$s24coverage_function_builder5test0SiyF"
 @Summer
 func test0() -> Int {
   // CHECK: [[@LINE-1]]:21 -> [[@LINE+3]]:2 : 0
   18
   12
 }
 
-// CHECK-LABEL: sil_coverage_map {{.*}} "$s24coverage_functon_builder5test1SiyF"
+// CHECK-LABEL: sil_coverage_map {{.*}} "$s24coverage_function_builder5test1SiyF"
 @Summer
 func test1() -> Int {
   // CHECK: [[@LINE-1]]:21 -> [[@LINE+7]]:2 : 0

test/Prototypes/DoubleWidth.swift.gyb

@@ -6,7 +6,7 @@ import StdlibUnittest
 
 // XXX: Auxiliary tests & benchmark for DoubleWidth were removed in PR #23024.
 // When the DoubleWidth makes comeback, revert the commits from there to
-// ressurect what was just a dead code in the meantime…
+// resurrect what was just a dead code in the meantime…
 
 /// A fixed-width integer that has twice the bit width of its base type.
 ///