llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-llvm-transforms Author: Stephan T. Lavavej (StephanTLavavej) <details> <summary>Changes</summary> After fixing a test name typo in #<!-- -->93259, I noticed these comment typos. (This is a separate "nice to have" PR, so that it doesn't get mixed up with the test changes that I need.) --- Full diff: https://github.com/llvm/llvm-project/pull/93260.diff 5 Files Affected: - (modified) flang/lib/Optimizer/HLFIR/IR/HLFIROps.cpp (+1-1) - (modified) flang/lib/Optimizer/HLFIR/Transforms/InlineElementals.cpp (+1-1) - (modified) lldb/include/lldb/Target/Process.h (+2-2) - (modified) llvm/lib/ExecutionEngine/Orc/TargetProcess/SimpleExecutorMemoryManager.cpp (+2-2) - (modified) llvm/test/Transforms/Coroutines/no-suspend.ll (+1-1) ``````````diff diff --git a/flang/lib/Optimizer/HLFIR/IR/HLFIROps.cpp b/flang/lib/Optimizer/HLFIR/IR/HLFIROps.cpp index 11196353b07c7..218b38e9ba79d 100644 --- a/flang/lib/Optimizer/HLFIR/IR/HLFIROps.cpp +++ b/flang/lib/Optimizer/HLFIR/IR/HLFIROps.cpp @@ -1115,7 +1115,7 @@ mlir::LogicalResult hlfir::MatmulOp::canonicalize(MatmulOp matmulOp, mlir::PatternRewriter &rewriter) { // the only two uses of the transposed matrix should be for the hlfir.matmul - // and hlfir.destory + // and hlfir.destroy auto isOtherwiseUnused = [&](hlfir::TransposeOp transposeOp) -> bool { std::size_t numUses = 0; for (mlir::Operation *user : transposeOp.getResult().getUsers()) { diff --git a/flang/lib/Optimizer/HLFIR/Transforms/InlineElementals.cpp b/flang/lib/Optimizer/HLFIR/Transforms/InlineElementals.cpp index 06d0518763848..6c8e3e1193747 100644 --- a/flang/lib/Optimizer/HLFIR/Transforms/InlineElementals.cpp +++ b/flang/lib/Optimizer/HLFIR/Transforms/InlineElementals.cpp @@ -32,7 +32,7 @@ namespace hlfir { } // namespace hlfir /// If the elemental has only two uses and those two are an apply operation and -/// a destory operation, return those two, otherwise return {} +/// a destroy operation, return those two, otherwise return {} static std::optional<std::pair<hlfir::ApplyOp, hlfir::DestroyOp>> getTwoUses(hlfir::ElementalOp elemental) { mlir::Operation::user_range users = elemental->getUsers(); diff --git a/lldb/include/lldb/Target/Process.h b/lldb/include/lldb/Target/Process.h index aac0cf51680a9..637d34c29715c 100644 --- a/lldb/include/lldb/Target/Process.h +++ b/lldb/include/lldb/Target/Process.h @@ -915,8 +915,8 @@ class Process : public std::enable_shared_from_this<Process>, /// \param[in] force_kill /// Whether lldb should force a kill (instead of a detach) from /// the inferior process. Normally if lldb launched a binary and - /// Destory is called, lldb kills it. If lldb attached to a - /// running process and Destory is called, lldb detaches. If + /// Destroy is called, lldb kills it. If lldb attached to a + /// running process and Destroy is called, lldb detaches. If /// this behavior needs to be over-ridden, this is the bool that /// can be used. /// diff --git a/llvm/lib/ExecutionEngine/Orc/TargetProcess/SimpleExecutorMemoryManager.cpp b/llvm/lib/ExecutionEngine/Orc/TargetProcess/SimpleExecutorMemoryManager.cpp index 4da031716e32a..3cdffb8cd0615 100644 --- a/llvm/lib/ExecutionEngine/Orc/TargetProcess/SimpleExecutorMemoryManager.cpp +++ b/llvm/lib/ExecutionEngine/Orc/TargetProcess/SimpleExecutorMemoryManager.cpp @@ -75,7 +75,7 @@ Error SimpleExecutorMemoryManager::finalize(tpctypes::FinalizeRequest &FR) { auto BailOut = [&](Error Err) { std::pair<void *, Allocation> AllocToDestroy; - // Get allocation to destory. + // Get allocation to destroy. { std::lock_guard<std::mutex> Lock(M); auto I = Allocations.find(Base.toPtr<void *>()); @@ -153,7 +153,7 @@ Error SimpleExecutorMemoryManager::deallocate( std::vector<std::pair<void *, Allocation>> AllocPairs; AllocPairs.reserve(Bases.size()); - // Get allocation to destory. + // Get allocation to destroy. Error Err = Error::success(); { std::lock_guard<std::mutex> Lock(M); diff --git a/llvm/test/Transforms/Coroutines/no-suspend.ll b/llvm/test/Transforms/Coroutines/no-suspend.ll index 53eb98f1273a9..fd8c5ac990958 100644 --- a/llvm/test/Transforms/Coroutines/no-suspend.ll +++ b/llvm/test/Transforms/Coroutines/no-suspend.ll @@ -325,7 +325,7 @@ body: %save = call token @llvm.coro.save(ptr %hdl) %subfn = call ptr @llvm.coro.subfn.addr(ptr %hdl, i8 1) call fastcc void %subfn(ptr %hdl) - ; memcpy separates destory from suspend, therefore cannot simplify. + ; memcpy separates destroy from suspend, therefore cannot simplify. call void @llvm.memcpy.p0.p0.i64(ptr %dst, ptr %src, i64 1, i1 false) %0 = call i8 @llvm.coro.suspend(token %save, i1 false) switch i8 %0, label %suspend [i8 0, label %resume `````````` </details> https://github.com/llvm/llvm-project/pull/93260 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
