Fix typos in comments of ExecutionEngine (NFC)

Reviewed By: sgraenitz

Differential Revision: https://reviews.llvm.org/D157226

Author: eymay

llvm/include/llvm-c/Orc.h

@@ -796,7 +796,7 @@ void LLVMOrcDisposeSymbols(LLVMOrcSymbolStringPoolEntryRef *Symbols);
  * method returns an error then clients should log it and call
  * LLVMOrcMaterializationResponsibilityFailMaterialization. If no dependencies
  * have been registered for the symbols covered by this
- * MaterializationResponsibiility then this method is guaranteed to return
+ * MaterializationResponsibility then this method is guaranteed to return
  * LLVMErrorSuccess.
  */
 LLVMErrorRef LLVMOrcMaterializationResponsibilityNotifyResolved(
@@ -813,7 +813,7 @@ LLVMErrorRef LLVMOrcMaterializationResponsibilityNotifyResolved(
  * method returns an error then clients should log it and call
  * LLVMOrcMaterializationResponsibilityFailMaterialization.
  * If no dependencies have been registered for the symbols covered by this
- * MaterializationResponsibiility then this method is guaranteed to return
+ * MaterializationResponsibility then this method is guaranteed to return
  * LLVMErrorSuccess.
  */
 LLVMErrorRef LLVMOrcMaterializationResponsibilityNotifyEmitted(
@@ -839,7 +839,7 @@ LLVMErrorRef LLVMOrcMaterializationResponsibilityDefineMaterializing(
 /**
  * Notify all not-yet-emitted covered by this MaterializationResponsibility
  * instance that an error has occurred.
- * This will remove all symbols covered by this MaterializationResponsibilty
+ * This will remove all symbols covered by this MaterializationResponsibility
  * from the target JITDylib, and send an error to any queries waiting on
  * these symbols.
  */

llvm/include/llvm/ExecutionEngine/ExecutionEngine.h

@@ -87,7 +87,7 @@ class ExecutionEngineState {
 
   /// Erase an entry from the mapping table.
   ///
-  /// \returns The address that \p ToUnmap was happed to.
+  /// \returns The address that \p ToUnmap was mapped to.
   uint64_t RemoveMapping(StringRef Name);
 };
 
@@ -261,7 +261,7 @@ class ExecutionEngine {
   /// locally can use the getFunctionAddress call, which will generate code
   /// and apply final preparations all in one step.
   ///
-  /// This method has no effect for the interpeter.
+  /// This method has no effect for the interpreter.
   virtual void generateCodeForModule(Module *M) {}
 
   /// finalizeObject - ensure the module is fully processed and is usable.
@@ -270,7 +270,7 @@ class ExecutionEngine {
   /// object usable for execution.  It should be called after sections within an
   /// object have been relocated using mapSectionAddress.  When this method is
   /// called the MCJIT execution engine will reapply relocations for a loaded
-  /// object.  This method has no effect for the interpeter.
+  /// object.  This method has no effect for the interpreter.
   ///
   /// Returns true on success, false on failure. Error messages can be retrieved
   /// by calling getError();

llvm/include/llvm/ExecutionEngine/JITEventListener.h

@@ -60,7 +60,7 @@ class JITEventListener {
   /// a previously emitted object is released.
   virtual void notifyFreeingObject(ObjectKey K) {}
 
-  // Get a pointe to the GDB debugger registration listener.
+  // Get a pointer to the GDB debugger registration listener.
   static JITEventListener *createGDBRegistrationListener();
 
 #if LLVM_USE_INTEL_JITEVENTS

llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h

@@ -623,7 +623,7 @@ class Symbol {
     this->S = static_cast<uint8_t>(S);
   }
 
-  /// Check wehther the given target flags are set for this Symbol.
+  /// Check whether the given target flags are set for this Symbol.
   bool hasTargetFlags(TargetFlagsType Flags) const {
     return static_cast<TargetFlagsType>(TargetFlags) & Flags;
   }

llvm/include/llvm/ExecutionEngine/JITSymbol.h

@@ -100,7 +100,7 @@ class JITSymbolFlags {
   JITSymbolFlags(FlagNames Flags, TargetFlagsType TargetFlags)
       : TargetFlags(TargetFlags), Flags(Flags) {}
 
-  /// Implicitly convert to bool. Returs true if any flag is set.
+  /// Implicitly convert to bool. Returns true if any flag is set.
   explicit operator bool() const { return Flags != None || TargetFlags != 0; }
 
   /// Compare for equality.

llvm/include/llvm/ExecutionEngine/OProfileWrapper.h

@@ -111,7 +111,7 @@ class OProfileWrapper {
   // daemon is running
   bool initialize();
 
-  // Searches /proc for the oprofile daemon and returns true if the process if
+  // Searches /proc for the oprofile daemon and returns true if the process is
   // found, or false otherwise.
   bool checkForOProfileProcEntry();
 

llvm/include/llvm/ExecutionEngine/Orc/COFFVCRuntimeSupport.h

@@ -33,7 +33,7 @@ class COFFVCRuntimeBootstrapper {
   /// Try to create a COFFVCRuntimeBootstrapper instance. An optional
   /// RuntimePath can be given to specify the location of directory that
   /// contains all vc runtime library files such as ucrt.lib and msvcrt.lib. If
-  /// not path was given, it will try to search the MSVC toolchain and Windows
+  /// no path was given, it will try to search the MSVC toolchain and Windows
   /// SDK installation and use the found library files automatically.
   ///
   /// Note that depending on the build setting, a different library
@@ -60,7 +60,7 @@ class COFFVCRuntimeBootstrapper {
   /// up COFFPlatform.
   Error initializeStaticVCRuntime(JITDylib &JD);
 
-  /// Adds symbol definitions of dynamic versino of msvc runtie libraries.
+  /// Adds symbol definitions of dynamic version of msvc runtime libraries.
   Expected<std::vector<std::string>>
   loadDynamicVCRuntime(JITDylib &JD, bool DebugVersion = false);
 

llvm/include/llvm/ExecutionEngine/Orc/CompileUtils.h

@@ -61,7 +61,7 @@ class SimpleCompiler : public IRCompileLayer::IRCompiler {
 
 /// A SimpleCompiler that owns its TargetMachine.
 ///
-/// This convenient for clients who don't want to own their TargetMachines,
+/// This is convenient for clients who don't want to own their TargetMachines,
 /// e.g. LLJIT.
 class TMOwningSimpleCompiler : public SimpleCompiler {
 public:

llvm/include/llvm/ExecutionEngine/Orc/Core.h

@@ -579,7 +579,7 @@ class MaterializationResponsibility {
   /// moved to the error state due to the failure of a dependency. If this
   /// method returns an error then clients should log it and call
   /// failMaterialize. If no dependencies have been registered for the
-  /// symbols covered by this MaterializationResponsibiility then this method
+  /// symbols covered by this MaterializationResponsibility then this method
   /// is guaranteed to return Error::success() and can be wrapped with cantFail.
   Error notifyResolved(const SymbolMap &Symbols);
 
@@ -591,7 +591,7 @@ class MaterializationResponsibility {
   /// moved to the error state due to the failure of a dependency. If this
   /// method returns an error then clients should log it and call
   /// failMaterialize. If no dependencies have been registered for the
-  /// symbols covered by this MaterializationResponsibiility then this method
+  /// symbols covered by this MaterializationResponsibility then this method
   /// is guaranteed to return Error::success() and can be wrapped with cantFail.
   Error notifyEmitted();
 
@@ -610,7 +610,7 @@ class MaterializationResponsibility {
 
   /// Notify all not-yet-emitted covered by this MaterializationResponsibility
   /// instance that an error has occurred.
-  /// This will remove all symbols covered by this MaterializationResponsibilty
+  /// This will remove all symbols covered by this MaterializationResponsibility
   /// from the target JITDylib, and send an error to any queries waiting on
   /// these symbols.
   void failMaterialization();
@@ -1674,7 +1674,7 @@ class ExecutionSession {
 
   /// Wrap a class method that takes concrete argument types (and a sender for
   /// a concrete return type) to produce an AsyncHandlerWrapperFunction. Uses
-  /// SPS to unpack teh arguments and pack the result.
+  /// SPS to unpack the arguments and pack the result.
   ///
   /// This function is intended to support easy construction of
   /// AsyncHandlerWrapperFunctions that can be associated with a tag

llvm/include/llvm/ExecutionEngine/Orc/DebugUtils.h

@@ -70,7 +70,7 @@ raw_ostream &operator<<(raw_ostream &OS, const MaterializationUnit &MU);
 raw_ostream &operator<<(raw_ostream &OS,
                         const JITDylibLookupFlags &JDLookupFlags);
 
-/// Rendar a SymbolLookupFlags instance.
+/// Render a SymbolLookupFlags instance.
 raw_ostream &operator<<(raw_ostream &OS, const SymbolLookupFlags &LookupFlags);
 
 /// Render a SymbolLookupSet entry.

llvm/include/llvm/ExecutionEngine/Orc/ELFNixPlatform.h

@@ -58,7 +58,7 @@ class ELFNixPlatform : public Platform {
   /// given JITDylib.
   ///
   /// The ORC runtime requires access to a number of symbols in
-  /// libc++. It is up to the caller to ensure that the requried
+  /// libc++. It is up to the caller to ensure that the required
   /// symbols can be referenced by code added to PlatformJD. The
   /// standard way to achieve this is to first attach dynamic library
   /// search generators for either the given process, or for the

llvm/include/llvm/ExecutionEngine/Orc/EPCGenericJITLinkMemoryManager.h

@@ -64,7 +64,7 @@ class EPCGenericJITLinkMemoryManager : public jitlink::JITLinkMemoryManager {
 namespace shared {
 
 /// FIXME: This specialization should be moved into TargetProcessControlTypes.h
-///        (or whereever those types get merged to) once ORC depends on JITLink.
+///        (or wherever those types get merged to) once ORC depends on JITLink.
 template <>
 class SPSSerializationTraits<SPSExecutorAddr,
                              jitlink::JITLinkMemoryManager::FinalizedAlloc> {

llvm/include/llvm/ExecutionEngine/Orc/ExecutorProcessControl.h

@@ -275,7 +275,7 @@ class ExecutorProcessControl {
 
   /// Search for symbols in the target process.
   ///
-  /// The result of the lookup is a 2-dimentional array of target addresses
+  /// The result of the lookup is a 2-dimensional array of target addresses
   /// that correspond to the lookup order. If a required symbol is not
   /// found then this method will return an error. If a weakly referenced
   /// symbol is not found then it be assigned a '0' value.

llvm/include/llvm/ExecutionEngine/Orc/IndirectionUtils.h

@@ -475,7 +475,7 @@ Expected<std::unique_ptr<JITCompileCallbackManager>>
 createLocalCompileCallbackManager(const Triple &T, ExecutionSession &ES,
                                   ExecutorAddr ErrorHandlerAddress);
 
-/// Create a local indriect stubs manager builder.
+/// Create a local indirect stubs manager builder.
 ///
 /// The given target triple will determine the ABI.
 std::function<std::unique_ptr<IndirectStubsManager>()>

llvm/include/llvm/ExecutionEngine/Orc/MachOPlatform.h

@@ -45,7 +45,7 @@ class MachOPlatform : public Platform {
   /// The ORC runtime requires access to a number of symbols in libc++, and
   /// requires access to symbols in libobjc, and libswiftCore to support
   /// Objective-C and Swift code. It is up to the caller to ensure that the
-  /// requried symbols can be referenced by code added to PlatformJD. The
+  /// required symbols can be referenced by code added to PlatformJD. The
   /// standard way to achieve this is to first attach dynamic library search
   /// generators for either the given process, or for the specific required
   /// libraries, to PlatformJD, then to create the platform instance:

llvm/include/llvm/ExecutionEngine/RTDyldMemoryManager.h

@@ -68,7 +68,7 @@ class RTDyldMemoryManager : public MCJITMemoryManager,
   /// Register EH frames in the current process.
   static void registerEHFramesInProcess(uint8_t *Addr, size_t Size);
 
-  /// Deregister EH frames in the current proces.
+  /// Deregister EH frames in the current process.
   static void deregisterEHFramesInProcess(uint8_t *Addr, size_t Size);
 
   void registerEHFrames(uint8_t *Addr, uint64_t LoadAddr, size_t Size) override;

llvm/lib/ExecutionEngine/JITLink/COFFLinkGraphBuilder.cpp

@@ -6,7 +6,7 @@
 //
 //===----------------------------------------------------------------------===//
 //
-// Generic COFF LinkGraph buliding code.
+// Generic COFF LinkGraph building code.
 //
 //===----------------------------------------------------------------------===//
 #include "COFFLinkGraphBuilder.h"
@@ -606,7 +606,7 @@ COFFLinkGraphBuilder::exportCOMDATSymbol(COFFSymbolIndex SymIndex,
                                          object::COFFSymbolRef Symbol) {
   Block *B = getGraphBlock(Symbol.getSectionNumber());
   auto &PendingComdatExport = PendingComdatExports[Symbol.getSectionNumber()];
-  // NOTE: ComdatDef->Legnth is the size of "section" not size of symbol.
+  // NOTE: ComdatDef->Length is the size of "section" not size of symbol.
   // We use zero symbol size to not reach out of bound of block when symbol
   // offset is non-zero.
   auto GSym = &G->addDefinedSymbol(

llvm/lib/ExecutionEngine/JITLink/ELFLinkGraphBuilder.cpp

@@ -6,7 +6,7 @@
 //
 //===----------------------------------------------------------------------===//
 //
-// Generic ELF LinkGraph buliding code.
+// Generic ELF LinkGraph building code.
 //
 //===----------------------------------------------------------------------===//
 

llvm/lib/ExecutionEngine/JITLink/ELFLinkGraphBuilder.h

@@ -51,7 +51,7 @@ class ELFLinkGraphBuilderBase {
   Section *CommonSection = nullptr;
 };
 
-/// Ling-graph building code that's specific to the given ELFT, but common
+/// LinkGraph building code that's specific to the given ELFT, but common
 /// across all architectures.
 template <typename ELFT>
 class ELFLinkGraphBuilder : public ELFLinkGraphBuilderBase {

llvm/lib/ExecutionEngine/JITLink/ELF_aarch64.cpp

@@ -598,7 +598,7 @@ void link_ELF_aarch64(std::unique_ptr<LinkGraph> G,
   PassConfiguration Config;
   const Triple &TT = G->getTargetTriple();
   if (Ctx->shouldAddDefaultTargetPasses(TT)) {
-    // Add eh-frame passses.
+    // Add eh-frame passes.
     Config.PrePrunePasses.push_back(DWARFRecordSectionSplitter(".eh_frame"));
     Config.PrePrunePasses.push_back(EHFrameEdgeFixer(
         ".eh_frame", 8, aarch64::Pointer32, aarch64::Pointer64,

llvm/lib/ExecutionEngine/JITLink/ELF_loongarch.cpp

@@ -186,7 +186,7 @@ void link_ELF_loongarch(std::unique_ptr<LinkGraph> G,
   PassConfiguration Config;
   const Triple &TT = G->getTargetTriple();
   if (Ctx->shouldAddDefaultTargetPasses(TT)) {
-    // Add eh-frame passses.
+    // Add eh-frame passes.
     Config.PrePrunePasses.push_back(DWARFRecordSectionSplitter(".eh_frame"));
     Config.PrePrunePasses.push_back(
         EHFrameEdgeFixer(".eh_frame", G->getPointerSize(), Pointer32, Pointer64,

llvm/lib/ExecutionEngine/JITLink/ELF_ppc64.cpp

@@ -471,7 +471,7 @@ void link_ELF_ppc64(std::unique_ptr<LinkGraph> G,
   if (Ctx->shouldAddDefaultTargetPasses(G->getTargetTriple())) {
     // Construct a JITLinker and run the link function.
 
-    // Add eh-frame passses.
+    // Add eh-frame passes.
     Config.PrePrunePasses.push_back(DWARFRecordSectionSplitter(".eh_frame"));
     Config.PrePrunePasses.push_back(EHFrameEdgeFixer(
         ".eh_frame", G->getPointerSize(), ppc64::Pointer32, ppc64::Pointer64,

llvm/lib/ExecutionEngine/JITLink/MachOLinkGraphBuilder.cpp

@@ -6,7 +6,7 @@
 //
 //===----------------------------------------------------------------------===//
 //
-// Generic MachO LinkGraph buliding code.
+// Generic MachO LinkGraph building code.
 //
 //===----------------------------------------------------------------------===//
 

llvm/lib/ExecutionEngine/JITLink/MachO_arm64.cpp

@@ -567,7 +567,7 @@ void link_MachO_arm64(std::unique_ptr<LinkGraph> G,
     Config.PrePrunePasses.push_back(
         CompactUnwindSplitter("__LD,__compact_unwind"));
 
-    // Add eh-frame passses.
+    // Add eh-frame passes.
     // FIXME: Prune eh-frames for which compact-unwind is available once
     // we support compact-unwind registration with libunwind.
     Config.PrePrunePasses.push_back(createEHFrameSplitterPass_MachO_arm64());

llvm/lib/ExecutionEngine/JITLink/MachO_x86_64.cpp

@@ -482,7 +482,7 @@ void link_MachO_x86_64(std::unique_ptr<LinkGraph> G,
   PassConfiguration Config;
 
   if (Ctx->shouldAddDefaultTargetPasses(G->getTargetTriple())) {
-    // Add eh-frame passses.
+    // Add eh-frame passes.
     Config.PrePrunePasses.push_back(createEHFrameSplitterPass_MachO_x86_64());
     Config.PrePrunePasses.push_back(createEHFrameEdgeFixerPass_MachO_x86_64());
 

llvm/lib/ExecutionEngine/Orc/Core.cpp

@@ -349,7 +349,7 @@ void ReExportsMaterializationUnit::materialize(
     }
   }
 
-  // The OnResolveInfo struct will hold the aliases and responsibilty for each
+  // The OnResolveInfo struct will hold the aliases and responsibility for each
   // query in the list.
   struct OnResolveInfo {
     OnResolveInfo(std::unique_ptr<MaterializationResponsibility> R,
@@ -2762,7 +2762,7 @@ void ExecutionSession::OL_completeLookup(
 
             // Otherwise this is a match.
 
-            // If this symbol is already in the requried state then notify the
+            // If this symbol is already in the required state then notify the
             // query, remove the symbol and continue.
             if (SymI->second.getState() >= Q->getRequiredState()) {
               LLVM_DEBUG(dbgs()

llvm/lib/ExecutionEngine/Orc/RTDyldObjectLinkingLayer.cpp

@@ -233,7 +233,7 @@ Error RTDyldObjectLinkingLayer::onObjLoad(
   if (auto *COFFObj = dyn_cast<object::COFFObjectFile>(&Obj)) {
     auto &ES = getExecutionSession();
 
-    // For all resolved symbols that are not already in the responsibilty set:
+    // For all resolved symbols that are not already in the responsibility set:
     // check whether the symbol is in a comdat section and if so mark it as
     // weak.
     for (auto &Sym : COFFObj->symbols()) {

llvm/lib/ExecutionEngine/Orc/SpeculateAnalyses.cpp

@@ -227,7 +227,7 @@ void SequenceBBQuery::traverseToExitBlock(const BasicBlock *AtBB,
                           VisitedBlocks);
 }
 
-// Get Block frequencies for blocks and take most frquently executed block,
+// Get Block frequencies for blocks and take most frequently executed block,
 // walk towards the entry block from those blocks and discover the basic blocks
 // with call.
 SequenceBBQuery::BlockListTy

llvm/lib/ExecutionEngine/PerfJITEvents/PerfJITEventListener.cpp

@@ -275,7 +275,7 @@ void PerfJITEventListener::notifyObjectLoaded(
             SectionIndex = SectOrErr.get()->getIndex();
 
     // According to spec debugging info has to come before loading the
-    // corresonding code load.
+    // corresponding code load.
     DILineInfoTable Lines = Context->getLineInfoForAddressRange(
         {*AddrOrErr, SectionIndex}, Size, FileLineInfoKind::AbsoluteFilePath);
 

llvm/lib/ExecutionEngine/RuntimeDyld/RTDyldMemoryManager.cpp

@@ -269,7 +269,7 @@ RTDyldMemoryManager::getSymbolAddressInProcess(const std::string &Name) {
 
   const char *NameStr = Name.c_str();
 
-  // DynamicLibrary::SearchForAddresOfSymbol expects an unmangled 'C' symbol
+  // DynamicLibrary::SearchForAddressOfSymbol expects an unmangled 'C' symbol
   // name so ff we're on Darwin, strip the leading '_' off.
 #ifdef __APPLE__
   if (NameStr[0] == '_')

llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldChecker.cpp

@@ -422,7 +422,7 @@ class RuntimeDyldCheckerExprEval {
     return std::make_pair(EvalResult(StubAddr), RemainingExpr);
   }
 
-  // Evaluate an identiefer expr, which may be a symbol, or a call to
+  // Evaluate an identifier expr, which may be a symbol, or a call to
   // one of the builtin functions: get_insn_opcode or get_insn_length.
   // Return the result, plus the expression remaining to be parsed.
   std::pair<EvalResult, StringRef> evalIdentifierExpr(StringRef Expr,
@@ -662,7 +662,7 @@ class RuntimeDyldCheckerExprEval {
     if (LHSResult.hasError() || RemainingExpr == "")
       return std::make_pair(LHSResult, RemainingExpr);
 
-    // Otherwise check if this is a binary expressioan.
+    // Otherwise check if this is a binary expression.
     BinOpToken BinOp;
     std::tie(BinOp, RemainingExpr) = parseBinOpToken(RemainingExpr);
 

llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.h

@@ -108,7 +108,7 @@ class RuntimeDyldELF : public RuntimeDyldImpl {
   uint64_t findOrAllocGOTEntry(const RelocationValueRef &Value,
                                unsigned GOTRelType);
 
-  // Resolve the relvative address of GOTOffset in Section ID and place
+  // Resolve the relative address of GOTOffset in Section ID and place
   // it at the given Offset
   void resolveGOTOffsetRelocation(unsigned SectionID, uint64_t Offset,
                                   uint64_t GOTOffset, uint32_t Type);
@@ -121,8 +121,8 @@ class RuntimeDyldELF : public RuntimeDyldImpl {
   // Compute the address in memory where we can find the placeholder
   void *computePlaceholderAddress(unsigned SectionID, uint64_t Offset) const;
 
-  // Split out common case for createing the RelocationEntry for when the relocation requires
-  // no particular advanced processing.
+  // Split out common case for creating the RelocationEntry for when the
+  // relocation requires no particular advanced processing.
   void processSimpleRelocation(unsigned SectionID, uint64_t Offset, unsigned RelType, RelocationValueRef Value);
 
   // Return matching *LO16 relocation (Mips specific)