Spelling tests

Closes https://github.com/dart-lang/sdk/pull/50920

GitOrigin-RevId: fa87531bd0f52b69485c9d02ff9e44a4a29c6a91
Change-Id: I0ae8574a5b77087895e004079f221201bb550cf3
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/278535
Reviewed-by: Martin Kustermann <kustermann@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
Commit-Queue: Alexander Thomas <athom@google.com>
diff --git a/pkg/analysis_server/test/lsp/completion_dart_test.dart b/pkg/analysis_server/test/lsp/completion_dart_test.dart
index 1624c59..8ff354d 100644
--- a/pkg/analysis_server/test/lsp/completion_dart_test.dart
+++ b/pkg/analysis_server/test/lsp/completion_dart_test.dart
@@ -1614,7 +1614,7 @@
     // but 'b'` to have its own.
     //
     // Additionally, because the caret is before the identifier, we will have
-    // seperate default insert/replace ranges.
+    // separate default insert/replace ranges.
     final content = '''
 void f(String a, {String? b}) {
   f([[^b]]);
diff --git a/pkg/compiler/doc/pragmas.md b/pkg/compiler/doc/pragmas.md
index 34b43ad..bf40596 100644
--- a/pkg/compiler/doc/pragmas.md
+++ b/pkg/compiler/doc/pragmas.md
@@ -246,5 +246,5 @@
 ```
 
 `dart2js:load-priority` annotations are _scoped_: when there are multiple
-annotations, the one on the nearest element enclosing the call to `loadLibary()`
-is in effect.
+annotations, the one on the nearest element enclosing the call to
+`loadLibrary()` is in effect.
diff --git a/pkg/compiler/lib/src/inferrer/node_tracer.dart b/pkg/compiler/lib/src/inferrer/node_tracer.dart
index 776e5c1..8e88c28 100644
--- a/pkg/compiler/lib/src/inferrer/node_tracer.dart
+++ b/pkg/compiler/lib/src/inferrer/node_tracer.dart
@@ -431,7 +431,7 @@
       bailout('Used as key in Map');
     }
 
-    // "a[...] = x" could be a list (container) or map assignemnt.
+    // "a[...] = x" could be a list (container) or map assignment.
     if (isIndexSetValue(info)) {
       var receiverType = info.receiver.type;
       if (inferrer.abstractValueDomain.isContainer(receiverType)) {
diff --git a/pkg/compiler/lib/src/inferrer_experimental/node_tracer.dart b/pkg/compiler/lib/src/inferrer_experimental/node_tracer.dart
index 3767b28..b60c228 100644
--- a/pkg/compiler/lib/src/inferrer_experimental/node_tracer.dart
+++ b/pkg/compiler/lib/src/inferrer_experimental/node_tracer.dart
@@ -420,7 +420,7 @@
       bailout('Used as key in Map');
     }
 
-    // "a[...] = x" could be a list (container) or map assignemnt.
+    // "a[...] = x" could be a list (container) or map assignment.
     if (isIndexSetValue(info)) {
       var receiverType = info.receiver.type;
       if (inferrer.abstractValueDomain.isContainer(receiverType)) {
diff --git a/pkg/compiler/lib/src/js_backend/namer.dart b/pkg/compiler/lib/src/js_backend/namer.dart
index ab16385..9ae3758 100644
--- a/pkg/compiler/lib/src/js_backend/namer.dart
+++ b/pkg/compiler/lib/src/js_backend/namer.dart
@@ -933,8 +933,8 @@
         name = 'lib_$name';
       }
     }
-    // Names constructed based on a libary name will be further disambiguated.
-    // However, as names from the same libary should have the same library
+    // Names constructed based on a library name will be further disambiguated.
+    // However, as names from the same library should have the same library
     // name part, we disambiguate the library name here.
     String disambiguated = name;
     for (int c = 0; libraryLongNames.containsValue(disambiguated); c++) {
diff --git a/pkg/compiler/lib/src/ssa/builder.dart b/pkg/compiler/lib/src/ssa/builder.dart
index 72250b7..678fe6f 100644
--- a/pkg/compiler/lib/src/ssa/builder.dart
+++ b/pkg/compiler/lib/src/ssa/builder.dart
@@ -1274,7 +1274,7 @@
   /// Builds an SSA graph for a sync*/async/async* generator.  We generate a
   /// entry function which tail-calls a body function. The entry contains
   /// per-invocation checks and the body, which is later transformed, contains
-  /// the re-entrant 'state machine' code.
+  /// the reentrant 'state machine' code.
   void _buildGenerator(FunctionEntity function, ir.FunctionNode functionNode) {
     _openFunction(function,
         functionNode: functionNode,
diff --git a/pkg/compiler/lib/src/ssa/type_builder.dart b/pkg/compiler/lib/src/ssa/type_builder.dart
index 3b7fe5a..b26eb6f 100644
--- a/pkg/compiler/lib/src/ssa/type_builder.dart
+++ b/pkg/compiler/lib/src/ssa/type_builder.dart
@@ -73,7 +73,7 @@
 
   /// Produces code that checks the runtime type is actually the type specified
   /// by attempting a type conversion.
-  HInstruction _checkBoolConverion(HInstruction original) {
+  HInstruction _checkBoolConversion(HInstruction original) {
     var checkInstruction =
         HBoolConversion(original, _abstractValueDomain.boolType);
     if (checkInstruction.isRedundant(_closedWorld)) {
@@ -154,7 +154,7 @@
     if (conditionCheckPolicy.isTrusted) {
       checkedOrTrusted = _trustType(original, boolType);
     } else if (conditionCheckPolicy.isEmitted) {
-      checkedOrTrusted = _checkBoolConverion(original);
+      checkedOrTrusted = _checkBoolConversion(original);
     }
     if (checkedOrTrusted == original) return original;
     builder.add(checkedOrTrusted);
diff --git a/pkg/compiler/test/codesize/swarm/swarm_ui_lib/observable/EventBatch.dart b/pkg/compiler/test/codesize/swarm/swarm_ui_lib/observable/EventBatch.dart
index 7eed98f..bd2272c 100644
--- a/pkg/compiler/test/codesize/swarm/swarm_ui_lib/observable/EventBatch.dart
+++ b/pkg/compiler/test/codesize/swarm/swarm_ui_lib/observable/EventBatch.dart
@@ -50,7 +50,7 @@
         return result;
       } else {
         // Already in a batch, so just use it.
-        // TODO(rnystrom): Re-entrant calls to wrap() are kind of hairy. They
+        // TODO(rnystrom): Reentrant calls to wrap() are kind of hairy. They
         // can occur in at least one known place:
         // 1. You respond to an event handler by calling a function with wrap()
         //    (i.e. the normal way we wrap event handlers).
diff --git a/pkg/compiler/test/codesize/swarm/update.py b/pkg/compiler/test/codesize/swarm/update.py
index fca04ea..45bd738 100755
--- a/pkg/compiler/test/codesize/swarm/update.py
+++ b/pkg/compiler/test/codesize/swarm/update.py
@@ -37,7 +37,7 @@
 
 
 def Flags():
-    """ Consturcts a parser for extracting flags from the command line. """
+    """ Constructs a parser for extracting flags from the command line. """
     result = optparse.OptionParser()
     result.add_option("-t",
                       "--target",
diff --git a/pkg/dart2js_info/CHANGELOG.md b/pkg/dart2js_info/CHANGELOG.md
index 6a617f6..0c47803 100644
--- a/pkg/dart2js_info/CHANGELOG.md
+++ b/pkg/dart2js_info/CHANGELOG.md
@@ -160,7 +160,7 @@
 - Moved `deferred_library_check` functionality to a library
 
 ## 0.2.2
-- Added `deferred_libary_check` tool
+- Added `deferred_library_check` tool
 
 ## 0.2.1
 - Merged `verify_deps` tool into `debug_info` tool
diff --git a/pkg/dart2wasm/lib/transformers.dart b/pkg/dart2wasm/lib/transformers.dart
index 6f11b1c..1d43a1b 100644
--- a/pkg/dart2wasm/lib/transformers.dart
+++ b/pkg/dart2wasm/lib/transformers.dart
@@ -309,7 +309,7 @@
   TreeNode _lowerAsyncStar(FunctionNode functionNode) {
     // TODO(joshualitt): This lowering is mostly reasonable, but if possible we
     // should try and figure out a way to remove the even / odd dance. That
-    // said, this will be replaced by an intrinsic implemntation ASAP so it may
+    // said, this will be replaced by an intrinsic implementation ASAP so it may
     // not be worth spending anymore time on this(aside from bug fixes).
     //
     // Transform
diff --git a/pkg/front_end/testcases/incremental/initializer_not_copied.yaml b/pkg/front_end/testcases/incremental/initializer_not_copied.yaml
index f34d811..1bee96c 100644
--- a/pkg/front_end/testcases/incremental/initializer_not_copied.yaml
+++ b/pkg/front_end/testcases/incremental/initializer_not_copied.yaml
@@ -3,7 +3,7 @@
 # BSD-style license that can be found in the LICENSE.md file.
 
 # Recompiling with no change shouldn't change the initializer.
-# In fact, this woukd (currently) print `[null]` when compiled once,
+# In fact, this would (currently) print `[null]` when compiled once,
 # but `[a1]` if recompiling main.dart first.
 # http://dartbug.com/48895
 
diff --git a/pkg/front_end/testcases/nnbd_mixed/const_is.dart b/pkg/front_end/testcases/nnbd_mixed/const_is.dart
index 9649a2d..e1b811c 100644
--- a/pkg/front_end/testcases/nnbd_mixed/const_is.dart
+++ b/pkg/front_end/testcases/nnbd_mixed/const_is.dart
@@ -15,7 +15,7 @@
   expect(true, test1);
 
   // void fn<T extends Null>() is void Function<T extends Never>()
-  // Should pass with weak checking because because Null becomes equivalent to
+  // Should pass with weak checking because Null becomes equivalent to
   // the bottom type.
   expect(true, fnWithNullBound is fnTypeWithNeverBound);
   const test2 = fnWithNullBound is fnTypeWithNeverBound;
diff --git a/pkg/js/proposal.md b/pkg/js/proposal.md
index c5eab45..8fe6d0d 100644
--- a/pkg/js/proposal.md
+++ b/pkg/js/proposal.md
@@ -1588,7 +1588,7 @@
 
 Dynamic dispatch, calling conventions, and runtime type checks, and native SDK
 types are different between the two compilers, so those details would require
-separete work.
+separate work.
 
 
 ### Q: If a JS API returns "Object" does this break dart2js tree shaking?
diff --git a/pkg/status_file/test/data/co19-dart2js.status b/pkg/status_file/test/data/co19-dart2js.status
index 9b778ac..70cc2e5 100644
--- a/pkg/status_file/test/data/co19-dart2js.status
+++ b/pkg/status_file/test/data/co19-dart2js.status
@@ -90,9 +90,9 @@
 Language/Expressions/Await_Expressions/evaluation_throws_t07: RuntimeError # Please triage this failure
 Language/Expressions/Constants/identifier_denotes_a_constant_t06: MissingCompileTimeError # Issue 26580
 Language/Expressions/Constants/identifier_denotes_a_constant_t07: MissingCompileTimeError # Issue 26580
-Language/Expressions/Function_Invocation/async_generator_invokation_t08: Skip # Issue 25967
-Language/Expressions/Function_Invocation/async_generator_invokation_t10: Skip # Issue 25967
-Language/Expressions/Function_Invocation/async_invokation_t04: RuntimeError, Pass # co19 issue 57
+Language/Expressions/Function_Invocation/async_generator_invocation_t08: Skip # Issue 25967
+Language/Expressions/Function_Invocation/async_generator_invocation_t10: Skip # Issue 25967
+Language/Expressions/Function_Invocation/async_invocation_t04: RuntimeError, Pass # co19 issue 57
 Language/Expressions/Instance_Creation/New/evaluation_t19: RuntimeError # Please triage this failure
 Language/Expressions/Instance_Creation/New/evaluation_t20: RuntimeError # Please triage this failure
 Language/Expressions/Maps/key_value_equals_operator_t02: CompileTimeError # Please triage this failure
@@ -1781,7 +1781,7 @@
 LibTest/html/Window/requestFileSystem_A01_t02: Skip # Roll 50 failure
 
 [ $compiler == dart2js && $runtime == chrome && $system == macos ]
-Language/Expressions/Function_Invocation/async_invokation_t04: RuntimeError # Please triage this failure
+Language/Expressions/Function_Invocation/async_invocation_t04: RuntimeError # Please triage this failure
 LayoutTests/fast/canvas/webgl/canvas-test_t01: Skip # Times out. Please triage this failure
 LayoutTests/fast/canvas/webgl/context-lost-restored_t01: Skip # Times out. Please triage this failure.
 LayoutTests/fast/canvas/webgl/draw-webgl-to-canvas-2d_t01: Pass, RuntimeError # Please triage this failure
diff --git a/pkg/test_runner/lib/src/browser_controller.dart b/pkg/test_runner/lib/src/browser_controller.dart
index 719a9e2..2a48778 100644
--- a/pkg/test_runner/lib/src/browser_controller.dart
+++ b/pkg/test_runner/lib/src/browser_controller.dart
@@ -1395,7 +1395,7 @@
       function reportError(msg) {
         function handleReady() {
           if (this.readyState == this.DONE && this.status != 200) {
-            var error = 'Sending back error did not succeeed: ' + this.status;
+            var error = 'Sending back error did not succeed: ' + this.status;
             error = error + '. Failed to send msg: ' + msg;
             error_div.innerHTML = error;
           }
diff --git a/pkg/vm_service/test/server_test.dart b/pkg/vm_service/test/server_test.dart
index 6cf1c38..eb8c978 100644
--- a/pkg/vm_service/test/server_test.dart
+++ b/pkg/vm_service/test/server_test.dart
@@ -277,7 +277,7 @@
         expect(
             responseQueue,
             emitsThrough(rpcErrorResponse(
-                RPCError('streamSubcribe', 103, 'Stream already subscribed', {
+                RPCError('streamSubscribe', 103, 'Stream already subscribed', {
               'details': "The stream '$streamId' is already subscribed",
             }))));
       }
diff --git a/pkg/vm_snapshot_analysis/lib/src/assets/D3SymbolTreeMap.js b/pkg/vm_snapshot_analysis/lib/src/assets/D3SymbolTreeMap.js
index 88dd692..38acb3e 100644
--- a/pkg/vm_snapshot_analysis/lib/src/assets/D3SymbolTreeMap.js
+++ b/pkg/vm_snapshot_analysis/lib/src/assets/D3SymbolTreeMap.js
@@ -349,7 +349,7 @@
         this.__cloneState.accepted + ' accepted on their own merits), ' +
         this.__cloneState.rejected + ' nodes (and their children) ' +
                                      'filtered out,' +
-        this.__cloneState.pruned + ' nodes pruned because because no ' +
+        this.__cloneState.pruned + ' nodes pruned because no ' +
                                    'children remained.');
     delete this.__cloneState;
   }
diff --git a/runtime/docs/compiler/type_testing_stubs.md b/runtime/docs/compiler/type_testing_stubs.md
index e7ddcec..b8a0a0d 100644
--- a/runtime/docs/compiler/type_testing_stubs.md
+++ b/runtime/docs/compiler/type_testing_stubs.md
@@ -64,7 +64,7 @@
 
 ## JIT mode
 
-In JIT mode we build the TTS lazily on first involcation (the TTS of types will be initialized to the `LazySpecializeTypeTest` stub). If later on more classes get loaded and the TTS fast path starts failing we re-build the TTS.
+In JIT mode we build the TTS lazily on first invocation (the TTS of types will be initialized to the `LazySpecializeTypeTest` stub). If later on more classes get loaded and the TTS fast path starts failing we re-build the TTS.
 
 After a hot-reload we reset all types to the lazy specialize TTS.
 
diff --git a/runtime/include/dart_tools_api.h b/runtime/include/dart_tools_api.h
index d43af24..42c2ba8 100644
--- a/runtime/include/dart_tools_api.h
+++ b/runtime/include/dart_tools_api.h
@@ -143,7 +143,7 @@
  * \return Returns a pointer to a Dart_EmbedderInformation structure.
  * The embedder keeps the ownership of the structure and any field in it.
  * The embedder must ensure that the structure will remain valid until the
- * next invokation of the callback.
+ * next invocation of the callback.
  */
 typedef void (*Dart_EmbedderInformationCallback)(
     Dart_EmbedderInformation* info);
diff --git a/runtime/observatory/tests/service/developer_service_get_isolate_id_test.dart b/runtime/observatory/tests/service/developer_service_get_isolate_id_test.dart
index 11b61da..ab5673e 100644
--- a/runtime/observatory/tests/service/developer_service_get_isolate_id_test.dart
+++ b/runtime/observatory/tests/service/developer_service_get_isolate_id_test.dart
@@ -67,12 +67,12 @@
     await initialIsolate.reload();
 
     // Grab the root library.
-    Service.Library rootLbirary =
+    Service.Library rootLibrary =
         await initialIsolate.rootLibrary.load() as Service.Library;
 
     // Grab self id.
     Service.Instance localSelfId = await initialIsolate.invoke(
-        rootLbirary, 'getSelfId') as Service.Instance;
+        rootLibrary, 'getSelfId') as Service.Instance;
 
     // Check that the id reported from dart:developer matches the id reported
     // from the service protocol.
@@ -81,7 +81,7 @@
 
     // Grab the child isolate's id.
     Service.Instance localChildId = await initialIsolate.invoke(
-        rootLbirary, 'getChildId') as Service.Instance;
+        rootLibrary, 'getChildId') as Service.Instance;
 
     // Check that the id reported from dart:developer matches the id reported
     // from the service protocol.
diff --git a/runtime/observatory_2/tests/service_2/developer_service_get_isolate_id_test.dart b/runtime/observatory_2/tests/service_2/developer_service_get_isolate_id_test.dart
index eb65cdc..4f40dc61 100644
--- a/runtime/observatory_2/tests/service_2/developer_service_get_isolate_id_test.dart
+++ b/runtime/observatory_2/tests/service_2/developer_service_get_isolate_id_test.dart
@@ -67,11 +67,11 @@
     await initialIsolate.reload();
 
     // Grab the root library.
-    Service.Library rootLbirary = await initialIsolate.rootLibrary.load();
+    Service.Library rootLibrary = await initialIsolate.rootLibrary.load();
 
     // Grab self id.
     Service.Instance localSelfId =
-        await initialIsolate.invoke(rootLbirary, 'getSelfId');
+        await initialIsolate.invoke(rootLibrary, 'getSelfId');
 
     // Check that the id reported from dart:developer matches the id reported
     // from the service protocol.
@@ -80,7 +80,7 @@
 
     // Grab the child isolate's id.
     Service.Instance localChildId =
-        await initialIsolate.invoke(rootLbirary, 'getChildId');
+        await initialIsolate.invoke(rootLibrary, 'getChildId');
 
     // Check that the id reported from dart:developer matches the id reported
     // from the service protocol.
diff --git a/runtime/platform/globals.h b/runtime/platform/globals.h
index 6c11173..2c9b740 100644
--- a/runtime/platform/globals.h
+++ b/runtime/platform/globals.h
@@ -674,7 +674,7 @@
   return destination;
 }
 
-// On Windows the reentrent version of strtok is called
+// On Windows the reentrant version of strtok is called
 // strtok_s. Unify on the posix name strtok_r.
 #if defined(DART_HOST_OS_WINDOWS)
 #define snprintf _sprintf_p
diff --git a/runtime/third_party/binary_size/src/template/D3SymbolTreeMap.js b/runtime/third_party/binary_size/src/template/D3SymbolTreeMap.js
index 88dd692..38acb3e 100644
--- a/runtime/third_party/binary_size/src/template/D3SymbolTreeMap.js
+++ b/runtime/third_party/binary_size/src/template/D3SymbolTreeMap.js
@@ -349,7 +349,7 @@
         this.__cloneState.accepted + ' accepted on their own merits), ' +
         this.__cloneState.rejected + ' nodes (and their children) ' +
                                      'filtered out,' +
-        this.__cloneState.pruned + ' nodes pruned because because no ' +
+        this.__cloneState.pruned + ' nodes pruned because no ' +
                                    'children remained.');
     delete this.__cloneState;
   }
diff --git a/runtime/vm/compiler/aot/precompiler.cc b/runtime/vm/compiler/aot/precompiler.cc
index 0698235..ed9a7bd 100644
--- a/runtime/vm/compiler/aot/precompiler.cc
+++ b/runtime/vm/compiler/aot/precompiler.cc
@@ -3548,7 +3548,7 @@
       // committed into the global object pool at the end of the compilation.
       // This makes an assumption that global object pool itself does not
       // grow during code generation - unfortunately this is not the case
-      // becase we might have nested code generation (i.e. we might generate
+      // because we might have nested code generation (i.e. we might generate
       // some stubs). If this indeed happens we retry the compilation.
       // (See TryCommitToParent invocation below).
       compiler::ObjectPoolBuilder object_pool_builder(
diff --git a/runtime/vm/compiler/backend/il.cc b/runtime/vm/compiler/backend/il.cc
index 77ad3b2..10b6951 100644
--- a/runtime/vm/compiler/backend/il.cc
+++ b/runtime/vm/compiler/backend/il.cc
@@ -3239,7 +3239,7 @@
 
   IntConverterInstr* box_defn = value()->definition()->AsIntConverter();
   if ((box_defn != NULL) && (box_defn->representation() == from())) {
-    // If the first convertion can erase bits (or deoptimize) we can't
+    // If the first conversion can erase bits (or deoptimize) we can't
     // canonicalize it away.
     auto src_defn = box_defn->value()->definition();
     if ((box_defn->from() == kUnboxedInt64) &&
diff --git a/runtime/vm/compiler/backend/il.h b/runtime/vm/compiler/backend/il.h
index 817a07a..60c6a72 100644
--- a/runtime/vm/compiler/backend/il.h
+++ b/runtime/vm/compiler/backend/il.h
@@ -10350,14 +10350,14 @@
   M(Arity, _, Name##Z, Inputs, Output)                                         \
   M(Arity, _, Name##W, Inputs, Output)
 
-// Define convertion between two SIMD types.
+// Define conversion between two SIMD types.
 #define SIMD_CONVERSION(M, FromType, ToType)                                   \
   M(1, _, FromType##To##ToType, (FromType), ToType)
 
 // List of all recognized SIMD operations.
 // Note: except for operations that map to operators (Add, Mul, Sub, Div,
 // BitXor, BitOr) all other operations must match names used by
-// MethodRecognizer. This allows to autogenerate convertion from
+// MethodRecognizer. This allows to autogenerate conversion from
 // MethodRecognizer::Kind into SimdOpInstr::Kind (see KindForMethod helper).
 // Note: M is for those SimdOp that are recognized methods and BINARY_OP
 // is for operators.
diff --git a/runtime/vm/compiler/backend/il_arm.cc b/runtime/vm/compiler/backend/il_arm.cc
index 4f093eb..80b2951 100644
--- a/runtime/vm/compiler/backend/il_arm.cc
+++ b/runtime/vm/compiler/backend/il_arm.cc
@@ -4761,7 +4761,7 @@
   }
 }
 
-DEFINE_EMIT(Simd32x4ToSimd32x4Convertion, (SameAsFirstInput, QRegister left)) {
+DEFINE_EMIT(Simd32x4ToSimd32x4Conversion, (SameAsFirstInput, QRegister left)) {
   // TODO(dartbug.com/30949) these operations are essentially nop and should
   // not generate any code. They should be removed from the graph before
   // code generation.
@@ -5107,7 +5107,7 @@
   ____(Float32x4Unary)                                                         \
   CASE(Float32x4ToInt32x4)                                                     \
   CASE(Int32x4ToFloat32x4)                                                     \
-  ____(Simd32x4ToSimd32x4Convertion)                                           \
+  ____(Simd32x4ToSimd32x4Conversion)                                           \
   SIMPLE(Float32x4Clamp)                                                       \
   SIMPLE(Float64x2Clamp)                                                       \
   CASE(Float32x4WithX)                                                         \
diff --git a/runtime/vm/compiler/backend/il_x64.cc b/runtime/vm/compiler/backend/il_x64.cc
index 6e88aa6..12ae30d 100644
--- a/runtime/vm/compiler/backend/il_x64.cc
+++ b/runtime/vm/compiler/backend/il_x64.cc
@@ -458,7 +458,7 @@
 LocationSummary* IfThenElseInstr::MakeLocationSummary(Zone* zone,
                                                       bool opt) const {
   comparison()->InitializeLocationSummary(zone, opt);
-  // TODO(dartbug.com/30952) support convertion of Register to corresponding
+  // TODO(dartbug.com/30952) support conversion of Register to corresponding
   // least significant byte register (e.g. RAX -> AL, RSI -> SIL, r15 -> r15b).
   comparison()->locs()->set_out(0, Location::RegisterLocation(RDX));
   return comparison()->locs();
diff --git a/runtime/vm/dart_api_impl_test.cc b/runtime/vm/dart_api_impl_test.cc
index 0730844..36b3111 100644
--- a/runtime/vm/dart_api_impl_test.cc
+++ b/runtime/vm/dart_api_impl_test.cc
@@ -10131,7 +10131,7 @@
 
 // There exists another test by name DartAPI_Invoke_CrossLibrary.
 // However, that currently fails for the dartk configuration as it
-// uses Dart_LoadLibray. This test here effectively tests the same
+// uses Dart_LoadLibrary. This test here effectively tests the same
 // functionality but invokes a function from an imported standard
 // library.
 TEST_CASE(DartAPI_InvokeImportedFunction) {
diff --git a/runtime/vm/dwarf.h b/runtime/vm/dwarf.h
index 492561e..fef33e4 100644
--- a/runtime/vm/dwarf.h
+++ b/runtime/vm/dwarf.h
@@ -98,7 +98,7 @@
     }
     Pair() : code(nullptr), value{} {}
 
-    // Don't implicitly delete copy and copy assigment constructors.
+    // Don't implicitly delete copy and copy assignment constructors.
     Pair(const Pair& other) = default;
     Pair& operator=(const Pair& other) = default;
 
diff --git a/runtime/vm/isolate_reload.h b/runtime/vm/isolate_reload.h
index 07361e0..42ea245 100644
--- a/runtime/vm/isolate_reload.h
+++ b/runtime/vm/isolate_reload.h
@@ -262,7 +262,7 @@
   BitVector* modified_libs_transitive_ = nullptr;
 
   // A bit vector indicating which of the saved libraries that transitively
-  // depend on a modified libary.
+  // depend on a modified library.
   BitVector* saved_libs_transitive_updated_ = nullptr;
 
   String& root_lib_url_;
diff --git a/runtime/vm/profiler.cc b/runtime/vm/profiler.cc
index 966cb84..24fab64 100644
--- a/runtime/vm/profiler.cc
+++ b/runtime/vm/profiler.cc
@@ -470,7 +470,7 @@
     // if we hit an assert while printing the stack.
     static RelaxedAtomic<uintptr_t> started_dump = 0;
     if (started_dump.fetch_add(1u) != 0) {
-      OS::PrintErr("Aborting re-entrant request for stack trace.\n");
+      OS::PrintErr("Aborting reentrant request for stack trace.\n");
       return;
     }
   }
diff --git a/runtime/vm/uri_test.cc b/runtime/vm/uri_test.cc
index 6c2bdd0..dd71655 100644
--- a/runtime/vm/uri_test.cc
+++ b/runtime/vm/uri_test.cc
@@ -476,7 +476,7 @@
 }
 
 // dart:core Uri allows for the base url to be relative (no scheme, no
-// authory, relative path) but this behavior is not in RFC 3986.  We
+// authority, relative path) but this behavior is not in RFC 3986.  We
 // do not implement this.
 TEST_CASE(ResolveUri_RelativeBase_NotImplemented) {
   const char* target_uri;
diff --git a/tests/co19_2/co19_2-dartdevc.status b/tests/co19_2/co19_2-dartdevc.status
index d927051..ea8e37b 100644
--- a/tests/co19_2/co19_2-dartdevc.status
+++ b/tests/co19_2/co19_2-dartdevc.status
@@ -57,8 +57,6 @@
 Language/Expressions/Constants/literal_number_t01: SkipByDesign # uses integer literal not representable as JavaScript number
 Language/Expressions/Constants/math_operators_t01: SkipByDesign # uses integer literal not representable as JavaScript number
 Language/Expressions/Constants/math_operators_t06: SkipByDesign # uses integer literal not representable as JavaScript number
-Language/Expressions/Function_Invocation/async_generator_invokation_t08: SkipSlow
-Language/Expressions/Function_Invocation/async_generator_invokation_t10: SkipSlow
 Language/Expressions/Numbers/integer_size_t03: SkipByDesign # uses integer literal not representable as JavaScript number
 Language/Expressions/Numbers/static_type_of_int_t01: SkipByDesign # uses integer literal not representable as JavaScript number
 Language/Expressions/Numbers/syntax_t06: SkipByDesign # uses integer literal not representable as JavaScript number
diff --git a/tests/corelib/uri_test.dart b/tests/corelib/uri_test.dart
index a1310c7..7d796a5 100644
--- a/tests/corelib/uri_test.dart
+++ b/tests/corelib/uri_test.dart
@@ -459,7 +459,7 @@
   }
 
   // Regression test, http://dartbug.com/20814
-  var uri = Uri.parse("/no-authorty/");
+  var uri = Uri.parse("/no-authority/");
   uri = uri.replace(fragment: "fragment");
   Expect.isFalse(uri.hasAuthority);
 
@@ -575,7 +575,7 @@
 
 void testBackslashes() {
   // Tests change which makes `\` be treated as `/` in
-  // autority and path.
+  // authority and path.
 
   Expect.stringEquals("https://example.com/",
       Uri.parse(r"https:\\example.com\").toString());
diff --git a/tests/corelib_2/uri_test.dart b/tests/corelib_2/uri_test.dart
index c78b483..e924905 100644
--- a/tests/corelib_2/uri_test.dart
+++ b/tests/corelib_2/uri_test.dart
@@ -461,7 +461,7 @@
   }
 
   // Regression test, http://dartbug.com/20814
-  var uri = Uri.parse("/no-authorty/");
+  var uri = Uri.parse("/no-authority/");
   uri = uri.replace(fragment: "fragment");
   Expect.isFalse(uri.hasAuthority);
 
@@ -577,7 +577,7 @@
 
 void testBackslashes() {
   // Tests change which makes `\` be treated as `/` in
-  // autority and path.
+  // authority and path.
 
   Expect.stringEquals("https://example.com/",
       Uri.parse(r"https:\\example.com\").toString());
diff --git a/tests/dartdevc/js_interop_test.dart b/tests/dartdevc/js_interop_test.dart
index bb27e5f..7015528 100644
--- a/tests/dartdevc/js_interop_test.dart
+++ b/tests/dartdevc/js_interop_test.dart
@@ -91,7 +91,7 @@
   final Function? localNullableLegacy = () => 'hello';
   final String Function()? localNullable = () => 'hello';
 
-  // Assigment to JS static field.
+  // Assignment to JS static field.
   Expect.throws(() {
     _jsStaticVariable = () => 'hello';
   });
diff --git a/tests/dartdevc_2/js_interop_test.dart b/tests/dartdevc_2/js_interop_test.dart
index 9ba148f..fafe310 100644
--- a/tests/dartdevc_2/js_interop_test.dart
+++ b/tests/dartdevc_2/js_interop_test.dart
@@ -89,7 +89,7 @@
   final Function localNonNullLegacy = () => 'hello';
   final String Function() localNonNull = () => 'hello';
 
-  // Assigment to JS static field.
+  // Assignment to JS static field.
   Expect.throws(() {
     _jsStaticVariable = () => 'hello';
   });
diff --git a/tests/language/async/identifier_test.dart b/tests/language/async/identifier_test.dart
index 0707259..2e6889b 100644
--- a/tests/language/async/identifier_test.dart
+++ b/tests/language/async/identifier_test.dart
@@ -27,7 +27,7 @@
 }
 
 main() {
-  // Each test declares a spearate async function, tests that `async`
+  // Each test declares a separate async function, tests that `async`
   // can occur in it, and makes sure the function is run.
   {
     const int async = 0;
diff --git a/tests/language/lazy/static8_test.dart b/tests/language/lazy/static8_test.dart
index de313a8..aefb751 100644
--- a/tests/language/lazy/static8_test.dart
+++ b/tests/language/lazy/static8_test.dart
@@ -4,7 +4,7 @@
 
 import "package:expect/expect.dart";
 
-// Test re-entrant initializer - calls throw StackOverflowError.
+// Test reentrant initializer - calls throw StackOverflowError.
 
 var trace;
 
diff --git a/tests/language/mixin_declaration/mixin_declaration_invalid_override_test.dart b/tests/language/mixin_declaration/mixin_declaration_invalid_override_test.dart
index 3eaa34c..e84d8af 100644
--- a/tests/language/mixin_declaration/mixin_declaration_invalid_override_test.dart
+++ b/tests/language/mixin_declaration/mixin_declaration_invalid_override_test.dart
@@ -56,7 +56,7 @@
 // Valid abstract class, super-invocation of foo hits implementation,
 // even if bar is still abstract.
 abstract class A5Foo = C5Foo with M5;
-// Invalid since super-invocaton of foo does not hit concrete implementation.
+// Invalid since super-invocation of foo does not hit concrete implementation.
 abstract class _ = C5Bar with M5;  //# 08: compile-time error
 
 class A5FooConcrete = A5Foo with C5Bar;
diff --git a/tests/language/nnbd/never/never_null_assignability_strong_test.dart b/tests/language/nnbd/never/never_null_assignability_strong_test.dart
index a902e45..8e9264a 100644
--- a/tests/language/nnbd/never/never_null_assignability_strong_test.dart
+++ b/tests/language/nnbd/never/never_null_assignability_strong_test.dart
@@ -9,7 +9,7 @@
 
 // Tests for direct calls to null safe functions.
 void testNullSafeCalls() {
-  // Test calling a null safe function expecting Null from a null safe libary
+  // Test calling a null safe function expecting Null from a null safe library
   {
     takesNull(nil);
     Expect.throws<String>(() => takesNull(never));
@@ -18,7 +18,7 @@
     Expect.throwsTypeError(() => (takesNull as dynamic)(3));
   }
 
-  // Test calling a null safe function expecting Never from a null safe libary
+  // Test calling a null safe function expecting Never from a null safe library
   {
     Expect.throws<String>(() => takesNever(never));
     // 3 can't be cast to Null or Never
@@ -26,7 +26,7 @@
     Expect.throwsTypeError(() => (takesNever as dynamic)(3));
   }
 
-  // Test calling a null safe function expecting int from a null safe libary
+  // Test calling a null safe function expecting int from a null safe library
   {
     takesInt(3);
     Expect.throwsTypeError(() => takesInt(nil as dynamic));
@@ -34,14 +34,14 @@
     Expect.throwsTypeError(() => (takesInt as dynamic)("hello"));
   }
 
-  // Test calling a null safe function expecting Object from a null safe libary
+  // Test calling a null safe function expecting Object from a null safe library
   {
     takesObject(3);
     Expect.throwsTypeError(() => takesObject(nil as dynamic));
     Expect.throwsTypeError(() => (takesObject as dynamic)(nil));
   }
 
-  // Test calling a null safe function expecting Object? from a null safe libary
+  // Test calling a null safe function expecting Object? from a null safe library
   {
     takesAny(3);
     takesAny(nil);
diff --git a/tests/language/nnbd/operator_type_test.dart b/tests/language/nnbd/operator_type_test.dart
index 497f0bb..fe677bd 100644
--- a/tests/language/nnbd/operator_type_test.dart
+++ b/tests/language/nnbd/operator_type_test.dart
@@ -3,7 +3,7 @@
 // BSD-style license that can be found in the LICENSE file.
 
 
-// Tests that language operators and constructrs requiring non-nullable values
+// Tests that language operators and constructs requiring non-nullable values
 // will not accept an operand with a nullable static type.
 // See: https://github.com/dart-lang/language/issues/298
 
diff --git a/tests/language/nnbd/subtyping/function_type_bounds_strong_test.dart b/tests/language/nnbd/subtyping/function_type_bounds_strong_test.dart
index 76e4c4e..ebacbf7 100644
--- a/tests/language/nnbd/subtyping/function_type_bounds_strong_test.dart
+++ b/tests/language/nnbd/subtyping/function_type_bounds_strong_test.dart
@@ -15,7 +15,7 @@
   Expect.isFalse(fnWithNonNullObjectBound is fnTypeWithNullableObjectBound);
 
   // void fn<T extends Null>() is void Function<T extends Never>()
-  // Should fail with strong checking because because Null and Never are treated
+  // Should fail with strong checking because Null and Never are treated
   // as distinct.
   Expect.isFalse(fnWithNullBound is fnTypeWithNeverBound);
 }
diff --git a/tests/language/nnbd/subtyping/function_type_bounds_weak_test.dart b/tests/language/nnbd/subtyping/function_type_bounds_weak_test.dart
index 976fdce..86b065a 100644
--- a/tests/language/nnbd/subtyping/function_type_bounds_weak_test.dart
+++ b/tests/language/nnbd/subtyping/function_type_bounds_weak_test.dart
@@ -17,7 +17,7 @@
   Expect.isTrue(test1);
 
   // void fn<T extends Null>() is void Function<T extends Never>()
-  // Should pass with weak checking because because Null becomes equivalent to
+  // Should pass with weak checking because Null becomes equivalent to
   // the bottom type.
   Expect.isTrue(fnWithNullBound is fnTypeWithNeverBound);
   const test2 = fnWithNullBound is fnTypeWithNeverBound;
diff --git a/tests/language/regress/regress32660_test.dart b/tests/language/regress/regress32660_test.dart
index 9ca6c0e62..081e4d2 100644
--- a/tests/language/regress/regress32660_test.dart
+++ b/tests/language/regress/regress32660_test.dart
@@ -3,7 +3,7 @@
 // BSD-style license that can be found in the LICENSE file.
 
 // noSuchMethod does not overwrite actual implementations, so if an
-// implemetation of a member exists that doesn't fulfill the interface it's
+// implementation of a member exists that doesn't fulfill the interface it's
 // an error.
 // On the other hand, if no implementation exists,
 // noSuchMethod will take its place and everything is okay.
diff --git a/tests/language/vm/load_to_load_forwarding_vm_test.dart b/tests/language/vm/load_to_load_forwarding_vm_test.dart
index d59cd57..69c830d 100644
--- a/tests/language/vm/load_to_load_forwarding_vm_test.dart
+++ b/tests/language/vm/load_to_load_forwarding_vm_test.dart
@@ -63,7 +63,7 @@
   return arr[0];
 }
 
-testPhiConvertions(f, arr) {
+testPhiConversions(f, arr) {
   if (f) {
     arr[0] = arr[1];
   } else {
@@ -559,8 +559,8 @@
   u32List[2] = 0x7FFFFFFF;
 
   for (var i = 0; i < 20; i++) {
-    testPhiConvertions(true, u32List);
-    testPhiConvertions(false, u32List);
+    testPhiConversions(true, u32List);
+    testPhiConversions(false, u32List);
   }
 
   for (var i = 0; i < 20; i++) {
diff --git a/tests/language/vm/type_vm_test.dart b/tests/language/vm/type_vm_test.dart
index a761309..14189b6 100644
--- a/tests/language/vm/type_vm_test.dart
+++ b/tests/language/vm/type_vm_test.dart
@@ -123,7 +123,7 @@
     return 1;
   }
 
-  static int testListAssigment() {
+  static int testListAssignment() {
     // Note: Many of these used to be runtime errors when implicit downcasts
     // were permitted. Now these are mostly compile time errors.
     int result = 0;
@@ -181,7 +181,7 @@
     Expect.equals(1, testFunctionNum());
     Expect.equals(9, testBoolCheck());
     Expect.equals(1, testFactory());
-    Expect.equals(8, testListAssigment());
+    Expect.equals(8, testListAssignment());
   }
 }
 
diff --git a/tests/language_2/async/identifier_test.dart b/tests/language_2/async/identifier_test.dart
index c53ca4f..c253ae7 100644
--- a/tests/language_2/async/identifier_test.dart
+++ b/tests/language_2/async/identifier_test.dart
@@ -29,7 +29,7 @@
 }
 
 main() {
-  // Each test declares a spearate async function, tests that `async`
+  // Each test declares a separate async function, tests that `async`
   // can occur in it, and makes sure the function is run.
   {
     const int async = 0;
diff --git a/tests/language_2/lazy/static8_test.dart b/tests/language_2/lazy/static8_test.dart
index faae6df..67f8adf 100644
--- a/tests/language_2/lazy/static8_test.dart
+++ b/tests/language_2/lazy/static8_test.dart
@@ -6,7 +6,7 @@
 
 import "package:expect/expect.dart";
 
-// Test re-entrant initializer - calls throw Error.
+// Test reentrant initializer - calls throw Error.
 
 var trace;
 
diff --git a/tests/language_2/mixin_declaration/mixin_declaration_invalid_override_test.dart b/tests/language_2/mixin_declaration/mixin_declaration_invalid_override_test.dart
index 45da93e..64dad10 100644
--- a/tests/language_2/mixin_declaration/mixin_declaration_invalid_override_test.dart
+++ b/tests/language_2/mixin_declaration/mixin_declaration_invalid_override_test.dart
@@ -58,7 +58,7 @@
 // Valid abstract class, super-invocation of foo hits implementation,
 // even if bar is still abstract.
 abstract class A5Foo = C5Foo with M5;
-// Invalid since super-invocaton of foo does not hit concrete implementation.
+// Invalid since super-invocation of foo does not hit concrete implementation.
 abstract class _ = C5Bar with M5;  //# 08: compile-time error
 
 class A5FooConcrete = A5Foo with C5Bar;
diff --git a/tests/language_2/regress/regress32660_test.dart b/tests/language_2/regress/regress32660_test.dart
index 090af81..8c00352 100644
--- a/tests/language_2/regress/regress32660_test.dart
+++ b/tests/language_2/regress/regress32660_test.dart
@@ -5,7 +5,7 @@
 // @dart = 2.9
 
 // noSuchMethod does not overwrite actual implementations, so if an
-// implemetation of a member exists that doesn't fulfill the interface it's
+// implementation of a member exists that doesn't fulfill the interface it's
 // an error.
 // On the other hand, if no implementation exists,
 // noSuchMethod will take its place and everything is okay.
diff --git a/tests/language_2/vm/load_to_load_forwarding_vm_test.dart b/tests/language_2/vm/load_to_load_forwarding_vm_test.dart
index e6606cf..fbbe30a 100644
--- a/tests/language_2/vm/load_to_load_forwarding_vm_test.dart
+++ b/tests/language_2/vm/load_to_load_forwarding_vm_test.dart
@@ -65,7 +65,7 @@
   return arr[0];
 }
 
-testPhiConvertions(f, arr) {
+testPhiConversions(f, arr) {
   if (f) {
     arr[0] = arr[1];
   } else {
@@ -561,8 +561,8 @@
   u32List[2] = 0x7FFFFFFF;
 
   for (var i = 0; i < 20; i++) {
-    testPhiConvertions(true, u32List);
-    testPhiConvertions(false, u32List);
+    testPhiConversions(true, u32List);
+    testPhiConversions(false, u32List);
   }
 
   for (var i = 0; i < 20; i++) {
diff --git a/tests/language_2/vm/type_vm_test.dart b/tests/language_2/vm/type_vm_test.dart
index a176a73..cd9ecee 100644
--- a/tests/language_2/vm/type_vm_test.dart
+++ b/tests/language_2/vm/type_vm_test.dart
@@ -124,7 +124,7 @@
     return 1;
   }
 
-  static int testListAssigment() {
+  static int testListAssignment() {
     int result = 0;
     {
       var a = new List<dynamic>.filled(5, null);
@@ -180,7 +180,7 @@
     Expect.equals(1, testFunctionNum());
     Expect.equals(9, testBoolCheck());
     Expect.equals(1, testFactory());
-    Expect.equals(8, testListAssigment());
+    Expect.equals(8, testListAssignment());
   }
 }
 
diff --git a/tests/lib/async/stream_state_helper.dart b/tests/lib/async/stream_state_helper.dart
index 4421075..2aded72 100644
--- a/tests/lib/async/stream_state_helper.dart
+++ b/tests/lib/async/stream_state_helper.dart
@@ -584,7 +584,7 @@
     return true;
   }
 
-  bool _testSubcribe() {
+  bool _testSubscribe() {
     _actual = "*[Subscribed]";
     return true;
   }
diff --git a/tests/lib/isolate/start_paused_test.dart b/tests/lib/isolate/start_paused_test.dart
index e7a0b84..5cb03e2 100644
--- a/tests/lib/isolate/start_paused_test.dart
+++ b/tests/lib/isolate/start_paused_test.dart
@@ -28,7 +28,7 @@
   // We start two isolates, one paused and one not.
   // The unpaused one must send an event, after which
   // we resume that paused isolate, and expect the second event.
-  // This is not a guaranteed test, since it can succeede even if the
+  // This is not a guaranteed test, since it can succeed even if the
   // paused isolate isn't really paused.
   // However, it must never fail, since that would mean that a paused
   // isolate sends a message.
diff --git a/tests/lib/math/math2_test.dart b/tests/lib/math/math2_test.dart
index 6802723..88f5159 100644
--- a/tests/lib/math/math2_test.dart
+++ b/tests/lib/math/math2_test.dart
@@ -110,7 +110,7 @@
     }
     if (b == 0.0) {
       // No need to look if they are close. Otherwise the check for 'a' above
-      // whould have triggered.
+      // would have triggered.
       Expect.equals(a, b);
     }
     final double shiftRightBy52 = 2.220446049250313080847263336181640625e-16;
diff --git a/tests/lib/math/math_test.dart b/tests/lib/math/math_test.dart
index ca32951..8ae09da 100644
--- a/tests/lib/math/math_test.dart
+++ b/tests/lib/math/math_test.dart
@@ -107,7 +107,7 @@
     }
     if (b == 0.0) {
       // No need to look if they are close. Otherwise the check for 'a' above
-      // whould have triggered.
+      // would have triggered.
       Expect.equals(a, b);
     }
     final double shiftRightBy52 = 2.220446049250313080847263336181640625e-16;
diff --git a/tests/lib_2/async/stream_state_helper.dart b/tests/lib_2/async/stream_state_helper.dart
index 46bf94d..131c058 100644
--- a/tests/lib_2/async/stream_state_helper.dart
+++ b/tests/lib_2/async/stream_state_helper.dart
@@ -583,7 +583,7 @@
     return true;
   }
 
-  bool _testSubcribe() {
+  bool _testSubscribe() {
     _actual = "*[Subscribed]";
     return true;
   }
diff --git a/tests/lib_2/isolate/start_paused_test.dart b/tests/lib_2/isolate/start_paused_test.dart
index e066a93..5a36249 100644
--- a/tests/lib_2/isolate/start_paused_test.dart
+++ b/tests/lib_2/isolate/start_paused_test.dart
@@ -30,7 +30,7 @@
   // We start two isolates, one paused and one not.
   // The unpaused one must send an event, after which
   // we resume that paused isolate, and expect the second event.
-  // This is not a guaranteed test, since it can succeede even if the
+  // This is not a guaranteed test, since it can succeed even if the
   // paused isolate isn't really paused.
   // However, it must never fail, since that would mean that a paused
   // isolate sends a message.
diff --git a/tests/lib_2/math/math2_test.dart b/tests/lib_2/math/math2_test.dart
index 9ec5071..8f8dc57 100644
--- a/tests/lib_2/math/math2_test.dart
+++ b/tests/lib_2/math/math2_test.dart
@@ -112,7 +112,7 @@
     }
     if (b == 0.0) {
       // No need to look if they are close. Otherwise the check for 'a' above
-      // whould have triggered.
+      // would have triggered.
       Expect.equals(a, b);
     }
     final double shiftRightBy52 = 2.220446049250313080847263336181640625e-16;
diff --git a/tests/lib_2/math/math_test.dart b/tests/lib_2/math/math_test.dart
index 5d2d71d..dccb5a3 100644
--- a/tests/lib_2/math/math_test.dart
+++ b/tests/lib_2/math/math_test.dart
@@ -109,7 +109,7 @@
     }
     if (b == 0.0) {
       // No need to look if they are close. Otherwise the check for 'a' above
-      // whould have triggered.
+      // would have triggered.
       Expect.equals(a, b);
     }
     final double shiftRightBy52 = 2.220446049250313080847263336181640625e-16;