feedback

Author: sbc100

lld/test/wasm/lto/stub-library.s

@@ -6,10 +6,10 @@
 # RUN: wasm-ld %t.o %t/libfoo.a %p/Inputs/stub.so -o %t.wasm
 # RUN: obj2yaml %t.wasm | FileCheck %s
 
-## The function `bar` is declared in stub.so and depends on `foo`, which happens
-## be in an LTO object, in an archive file.
-## This verifies that stub library dependencies (required exports) can be defined
-## in LTO objects.
+## The function `bar` is declared in stub.so and depends on `foo`, which in this
+## case is defined in an LTO object, inside an archive file.
+## This verifies that stub library dependencies (required exports) can be
+## defined in LTO objects.
 .functype bar () -> ()
 
 .globl _start

lld/wasm/Driver.cpp

@@ -949,11 +949,11 @@ static void processStubLibrariesPreLTO() {
           auto* needed = symtab->find(dep);
           if (needed ) {
             needed->isUsedInRegularObj = true;
-            // Like with handleLibcall we have extract any LTO archive members
-            // that might need to be exported due to stub library symbol being
-            // used.  Without this the LTO object could be extracted during
-            // processStubLibraries, which is too late since LTO has already
-            // beeing performed at that point.
+            // Like with handleLibcall we have to extract any LTO archive
+            // members that might need to be exported due to stub library
+            // symbols being referenced.  Without this the LTO object could be
+            // extracted during processStubLibraries, which is too late since
+            // LTO has already being performed at that point.
             if (needed->isLazy() && isa<BitcodeFile>(needed->getFile())) {
               if (!config->whyExtract.empty())
                 ctx.whyExtractRecords.emplace_back(toString(stub_file),