[SwiftCompilerSources] Fix typos

Author: Sajjon

SwiftCompilerSources/Sources/Optimizer/FunctionPasses/AsyncDemotion.swift

@@ -590,7 +590,7 @@ private func analyzeForDemotion(_ function: Function) -> Node? {
   // because it may be a caller of a function that can be demoted.
 
   var executors = Set<Executor>()
-  var knownAsyncApplys: [ApplySite] = []
+  var knownAsyncApplySites: [ApplySite] = []
   var hops: [Instruction] = []
   var unknownAsyncOp: Instruction? = nil
 
@@ -614,7 +614,7 @@ private func analyzeForDemotion(_ function: Function) -> Node? {
       if apply.referencedFunction == nil {
         unknownAsyncOp.setIfUnset(inst)
       } else {
-        knownAsyncApplys.append(apply)
+        knownAsyncApplySites.append(apply)
         stats.tick(.asyncKnownCallsCount)
       }
 
@@ -630,7 +630,7 @@ private func analyzeForDemotion(_ function: Function) -> Node? {
 
   let data: AnalysisResult.Data =
     (function: function,
-     knownAsyncCalls: knownAsyncApplys,
+     knownAsyncCalls: knownAsyncApplySites,
      executors: executors,
      hops: hops)
 

SwiftCompilerSources/Sources/Optimizer/FunctionPasses/ComputeSideEffects.swift

@@ -145,7 +145,7 @@ private struct CollectedEffects {
       // In addition to the effects of the apply, also consider the
       // effects of the capture, which reads the captured value in
       // order to move it into the context. This only applies to
-      // addressible values, because capturing does not dereference
+      // addressable values, because capturing does not dereference
       // any class objects.
       //
       // Ignore captures for on-stack partial applies. They only
@@ -332,7 +332,7 @@ private struct CollectedEffects {
   /// Adds effects to a specific value.
   ///
   /// If the value comes from an argument (or multiple arguments), then the effects are added
-  /// to the corrseponding `argumentEffects`. Otherwise they are added to the `global` effects.
+  /// to the corresponding `argumentEffects`. Otherwise they are added to the `global` effects.
   private mutating func addEffects(_ effects: SideEffects.GlobalEffects, to value: Value) {
     addEffects(effects, to: value, fromInitialPath: defaultPath(for: value))
   }

SwiftCompilerSources/Sources/Optimizer/FunctionPasses/LifetimeDependenceScopeFixup.swift

@@ -112,7 +112,7 @@ private func extendAccessScopes(dependence: LifetimeDependence,
 /// caller scope, which is handled separately). A nested 'read' access can never interfere with another access in the
 /// same outer 'read', because it is impossible to nest a 'modify' access within a 'read'. For 'modify' accesses,
 /// however, the inner scope must be extended for correctness. A 'modify' access can interfere with other 'modify'
-/// accesss in the same scope. We rely on exclusivity diagnostics to report these interferences. For example:
+/// access in the same scope. We rely on exclusivity diagnostics to report these interferences. For example:
 ///
 ///     sil @foo : $(@inout C) -> () {
 ///       bb0(%0 : $*C):
@@ -133,12 +133,12 @@ private func extendAccessScopes(dependence: LifetimeDependence,
 /// violation, and that subsequent optimizations do not shrink the inner access `%a1`.
 private func extendAccessScope(beginAccess: BeginAccessInst, range: inout InstructionRange,
                                _ context: FunctionPassContext) -> FunctionArgument? {
-  var endAcceses = [Instruction]()
+  var endAccesses = [Instruction]()
   // Collect the original end_access instructions and extend the range to to cover them. The resulting access scope must
   // cover the original scope because it may protect other memory operations.
   var requiresExtension = false
   for end in beginAccess.endInstructions {
-    endAcceses.append(end)
+    endAccesses.append(end)
     if range.contains(end) {
       // If any end_access is inside the new range, then all end_accesses must be rewritten.
       requiresExtension = true
@@ -171,7 +171,7 @@ private func extendAccessScope(beginAccess: BeginAccessInst, range: inout Instru
     range.insert(endAccess)
   }
   // Delete original end_access instructions
-  for endAccess in endAcceses {
+  for endAccess in endAccesses {
     context.erase(instruction: endAccess)
   }
   // TODO: Add SIL support for lifetime dependence and write unit test for nested access scopes

SwiftCompilerSources/Sources/Optimizer/FunctionPasses/StackPromotion.swift

@@ -273,7 +273,7 @@ private struct ComputeOuterBlockrange : EscapeVisitorWithResult {
     // instructions (for which the `visitUse` closure is not called).
     result.insert(operandsDefinitionBlock)
 
-    // We need to explicitly add predecessor blocks of phis becaues they
+    // We need to explicitly add predecessor blocks of phis because they
     // are not necesesarily visited during the down-walk in `isEscaping()`.
     // This is important for the special case where there is a back-edge from the
     // inner range to the inner rage's begin-block:

SwiftCompilerSources/Sources/Optimizer/InstructionSimplification/SimplifyBeginBorrow.swift

@@ -132,7 +132,7 @@ private extension Instruction {
     // In instruction simplification we don't have a domtree. Therefore do a simple dominance
     // check based on same-block relations.
     if parentBlock == value.parentBlock {
-      // The value and instruction are in the same block. All uses are dominanted by both.
+      // The value and instruction are in the same block. All uses are dominated by both.
       return true
     }
     let destroys = value.uses.filterUsers(ofType: DestroyValueInst.self)

SwiftCompilerSources/Sources/Optimizer/InstructionSimplification/SimplifyLoad.swift

@@ -20,7 +20,7 @@ extension LoadInst : OnoneSimplifyable, SILCombineSimplifyable {
     if optimizeLoadFromStringLiteral(context) {
       return
     }
-    if optmizeLoadFromEmptyCollection(context) {
+    if optimizeLoadFromEmptyCollection(context) {
       return
     }
     if replaceLoadOfGlobalLet(context) {
@@ -85,7 +85,7 @@ extension LoadInst : OnoneSimplifyable, SILCombineSimplifyable {
 
   /// Loading `count` or `capacity` from the empty `Array`, `Set` or `Dictionary` singleton
   /// is replaced by a 0 literal.
-  private func optmizeLoadFromEmptyCollection(_ context: SimplifyContext) -> Bool {
+  private func optimizeLoadFromEmptyCollection(_ context: SimplifyContext) -> Bool {
     if self.isZeroLoadFromEmptyCollection() {
       let builder = Builder(before: self, context)
       let zeroLiteral = builder.createIntegerLiteral(0, type: type)

SwiftCompilerSources/Sources/Optimizer/PassManager/Context.swift

@@ -184,7 +184,7 @@ extension MutatingContext {
   }
 
   func inlineFunction(apply: FullApplySite, mandatoryInline: Bool) {
-    // This is only a best-effort attempt to notity the new cloned instructions as changed.
+    // This is only a best-effort attempt to notify the new cloned instructions as changed.
     // TODO: get a list of cloned instructions from the `inlineFunction`
     let instAfterInling: Instruction?
     switch apply {

SwiftCompilerSources/Sources/Optimizer/Utilities/AddressUtils.swift

@@ -194,7 +194,7 @@ extension AddressUseVisitor {
       if operand.instruction.isIncidentalUse {
         return leafAddressUse(of: operand)
       }
-      // Unkown instruction.
+      // Unknown instruction.
       return unknownAddressUse(of: operand)
     }
   }
@@ -289,7 +289,7 @@ extension AddressInitializationWalker {
   }
 }
 
-// Implement AddresUseVisitor
+// Implement AddressUseVisitor
 extension AddressInitializationWalker {
   /// An address projection produces a single address result and does not
   /// escape its address operand in any other way.
@@ -359,7 +359,7 @@ extension AddressInitializationWalker {
   }
 }
 
-/// A live range representing the ownership of addressible memory.
+/// A live range representing the ownership of addressable memory.
 ///
 /// This live range represents the minimal guaranteed lifetime of the object being addressed. Uses of derived addresses
 /// may be extended up to the ends of this scope without violating ownership.

SwiftCompilerSources/Sources/Optimizer/Utilities/BorrowUtils.swift

@@ -126,7 +126,7 @@
 //
 //
 // TODO: These utilities should be integrated with OSSA SIL verification and
-// guaranteed to be compelete (produce known results for all legal SIL
+// guaranteed to be complete (produce known results for all legal SIL
 // patterns).
 // ===----------------------------------------------------------------------===//
 
@@ -211,7 +211,7 @@ enum BorrowingInstruction : CustomStringConvertible, Hashable {
   /// the last in the function (e.g. a store rather than a destroy or return).
   /// The client needs to use LifetimeDependenceDefUseWalker to do better.
   ///
-  /// TODO: to hande reborrow-extended uses, migrate ExtendedLiveness
+  /// TODO: to handle reborrow-extended uses, migrate ExtendedLiveness
   /// to SwiftCompilerSources.
   ///
   /// TODO: Handle .partialApply and .markDependence forwarded uses
@@ -278,7 +278,7 @@ enum BorrowingInstruction : CustomStringConvertible, Hashable {
 ///
 /// If the value is a begin_apply result, then it may be the token or
 /// one of the yielded values. In any case, the scope ending operands
-/// are on the end_apply or abort_apply intructions that use the
+/// are on the end_apply or abort_apply instructions that use the
 /// token.
 ///
 /// Note: equivalent to C++ BorrowedValue, but also handles begin_apply.

SwiftCompilerSources/Sources/Optimizer/Utilities/EscapeUtils.swift

@@ -156,7 +156,7 @@ protocol EscapeVisitor {
   /// If true, the traversals follow values with trivial types.
   var followTrivialTypes: Bool { get }
 
-  /// If true, the traveral follows loaded values.
+  /// If true, the traversal follows loaded values.
   var followLoads: Bool { get }
 }