[lld] 9f90347 - fix comment typos to cycle bots

Nico Weber via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 27 06:53:17 PDT 2021


Author: Nico Weber
Date: 2021-10-27T09:53:08-04:00
New Revision: 9f9034758896530c97861392ba49c4847347b807

URL: https://github.com/llvm/llvm-project/commit/9f9034758896530c97861392ba49c4847347b807
DIFF: https://github.com/llvm/llvm-project/commit/9f9034758896530c97861392ba49c4847347b807.diff

LOG: fix comment typos to cycle bots

Added: 
    

Modified: 
    lld/wasm/Config.h
    lld/wasm/Driver.cpp
    lld/wasm/InputChunks.cpp
    lld/wasm/InputFiles.cpp
    lld/wasm/Options.td
    lld/wasm/Relocations.cpp
    lld/wasm/SymbolTable.cpp
    lld/wasm/Symbols.cpp
    lld/wasm/Symbols.h
    lld/wasm/Writer.cpp

Removed: 
    


################################################################################
diff  --git a/lld/wasm/Config.h b/lld/wasm/Config.h
index 7b20a6b03854f..08c21f3b18456 100644
--- a/lld/wasm/Config.h
+++ b/lld/wasm/Config.h
@@ -77,7 +77,7 @@ struct Configuration {
   llvm::Optional<std::vector<std::string>> features;
 
   // The following config options do not directly correspond to any
-  // particualr command line options.
+  // particular command line options.
 
   // True if we are creating position-independent code.
   bool isPic;

diff  --git a/lld/wasm/Driver.cpp b/lld/wasm/Driver.cpp
index 4f7423c06e832..fe90876451f31 100644
--- a/lld/wasm/Driver.cpp
+++ b/lld/wasm/Driver.cpp
@@ -684,7 +684,7 @@ static void createOptionalSymbols() {
 
   // For non-shared memory programs we still need to define __tls_base since we
   // allow object files built with TLS to be linked into single threaded
-  // programs, and such object files can contains refernced to this symbol.
+  // programs, and such object files can contain references to this symbol.
   //
   // However, in this case __tls_base is immutable and points directly to the
   // start of the `.tdata` static segment.

diff  --git a/lld/wasm/InputChunks.cpp b/lld/wasm/InputChunks.cpp
index 1207f523f2ee4..7973dce6827e2 100644
--- a/lld/wasm/InputChunks.cpp
+++ b/lld/wasm/InputChunks.cpp
@@ -355,7 +355,7 @@ uint64_t InputChunk::getVA(uint64_t offset) const {
 }
 
 // Generate code to apply relocations to the data section at runtime.
-// This is only called when generating shared libaries (PIC) where address are
+// This is only called when generating shared libraries (PIC) where address are
 // not known at static link time.
 void InputChunk::generateRelocationCode(raw_ostream &os) const {
   LLVM_DEBUG(dbgs() << "generating runtime relocations: " << getName()

diff  --git a/lld/wasm/InputFiles.cpp b/lld/wasm/InputFiles.cpp
index 29bc0b32d9658..58da9cce08152 100644
--- a/lld/wasm/InputFiles.cpp
+++ b/lld/wasm/InputFiles.cpp
@@ -171,7 +171,7 @@ uint64_t ObjFile::calcNewValue(const WasmRelocation &reloc, uint64_t tombstone,
       return 0;
     auto D = cast<DefinedData>(sym);
     // Treat non-TLS relocation against symbols that live in the TLS segment
-    // like TLS relocations.  This beaviour exists to support older object
+    // like TLS relocations.  This behaviour exists to support older object
     // files created before we introduced TLS relocations.
     // TODO(sbc): Remove this legacy behaviour one day.  This will break
     // backward compat with old object files built with `-fPIC`.

diff  --git a/lld/wasm/Options.td b/lld/wasm/Options.td
index 40cb314c4ab09..79d62b7c69fbd 100644
--- a/lld/wasm/Options.td
+++ b/lld/wasm/Options.td
@@ -141,8 +141,9 @@ def z: JoinedOrSeparate<["-"], "z">, MetaVarName<"<option>">,
 // The follow flags are unique to wasm
 
 def allow_undefined: F<"allow-undefined">,
-  HelpText<"Allow undefined symbols in linked binary. This options is equivelant "
-           "to --import-undefined and --unresolved-symbols=ignore-all">;
+  HelpText<"Allow undefined symbols in linked binary. "
+           "This options is equivalent to --import-undefined "
+           "and --unresolved-symbols=ignore-all">;
 
 def import_undefined: F<"import-undefined">,
   HelpText<"Turn undefined symbols into imports where possible">;

diff  --git a/lld/wasm/Relocations.cpp b/lld/wasm/Relocations.cpp
index 847ce87b4aa03..b0acd66465aeb 100644
--- a/lld/wasm/Relocations.cpp
+++ b/lld/wasm/Relocations.cpp
@@ -154,7 +154,7 @@ void scanRelocations(InputChunk *chunk) {
       case R_WASM_MEMORY_ADDR_I64:
         // These relocation types are only present in the data section and
         // will be converted into code by `generateRelocationCode`.  This code
-        // requires the symbols to have GOT entires.
+        // requires the symbols to have GOT entries.
         if (requiresGOTAccess(sym))
           addGOTEntry(sym);
         break;

diff  --git a/lld/wasm/SymbolTable.cpp b/lld/wasm/SymbolTable.cpp
index b204fae451a94..098d2e01efd62 100644
--- a/lld/wasm/SymbolTable.cpp
+++ b/lld/wasm/SymbolTable.cpp
@@ -144,7 +144,7 @@ static bool signatureMatches(FunctionSymbol *existing,
                              const WasmSignature *newSig) {
   const WasmSignature *oldSig = existing->signature;
 
-  // If either function is missing a signature (this happend for bitcode
+  // If either function is missing a signature (this happens for bitcode
   // symbols) then assume they match.  Any mismatch will be reported later
   // when the LTO objects are added.
   if (!newSig || !oldSig)

diff  --git a/lld/wasm/Symbols.cpp b/lld/wasm/Symbols.cpp
index 79bec1025a254..39c91a11e20e7 100644
--- a/lld/wasm/Symbols.cpp
+++ b/lld/wasm/Symbols.cpp
@@ -223,7 +223,7 @@ void Symbol::setHidden(bool isHidden) {
 
 bool Symbol::isExported() const {
   // Shared libraries must export all weakly defined symbols
-  // in case they contain the version that will be chosed by
+  // in case they contain the version that will be chosen by
   // the dynamic linker.
   if (config->shared && isLive() && isDefined() && isWeak())
     return true;

diff  --git a/lld/wasm/Symbols.h b/lld/wasm/Symbols.h
index 342ebcf1a6e9f..31e4fa4805a81 100644
--- a/lld/wasm/Symbols.h
+++ b/lld/wasm/Symbols.h
@@ -153,7 +153,7 @@ class Symbol {
   // are unreferenced except by other bitcode objects.
   bool isUsedInRegularObj : 1;
 
-  // True if ths symbol is explicitly marked for export (i.e. via the
+  // True if this symbol is explicitly marked for export (i.e. via the
   // -e/--export command line flag)
   bool forceExport : 1;
 

diff  --git a/lld/wasm/Writer.cpp b/lld/wasm/Writer.cpp
index 454daaf34403f..2efa036cf1c67 100644
--- a/lld/wasm/Writer.cpp
+++ b/lld/wasm/Writer.cpp
@@ -1227,7 +1227,7 @@ void Writer::createApplyDataRelocationsFunction() {
 }
 
 // Similar to createApplyDataRelocationsFunction but generates relocation code
-// fro WebAssembly globals. Because these globals are not shared between threads
+// for WebAssembly globals. Because these globals are not shared between threads
 // these relocation need to run on every thread.
 void Writer::createApplyGlobalRelocationsFunction() {
   // First write the body's contents to a string.
@@ -1466,8 +1466,8 @@ void Writer::run() {
       sym->forceExport = true;
   }
 
-  // Delay reporting error about explict exports until after addStartStopSymbols
-  // which can create optional symbols.
+  // Delay reporting error about explicit exports until after
+  // addStartStopSymbols which can create optional symbols.
   for (auto &name : config->requiredExports) {
     Symbol *sym = symtab->find(name);
     if (!sym || !sym->isDefined()) {
@@ -1479,7 +1479,7 @@ void Writer::run() {
   }
 
   if (config->isPic && !config->sharedMemory) {
-    // In shared memory mode all data segments are passive and initilized
+    // In shared memory mode all data segments are passive and initialized
     // via __wasm_init_memory.
     log("-- combineOutputSegments");
     combineOutputSegments();


        


More information about the llvm-commits mailing list