Skip to content

[NFC][offload] Resolve warning about pessimized copy ellision.#152146

Closed
maksimsab wants to merge 1 commit into
llvm:mainfrom
maksimsab:remove_move
Closed

[NFC][offload] Resolve warning about pessimized copy ellision.#152146
maksimsab wants to merge 1 commit into
llvm:mainfrom
maksimsab:remove_move

Conversation

@maksimsab
Copy link
Copy Markdown
Contributor

This patch resolves post-commit failure after merging #131347.

This patch resolves post-commit failure after merging llvm#131347.
@llvmbot
Copy link
Copy Markdown
Member

llvmbot commented Aug 5, 2025

@llvm/pr-subscribers-llvm-transforms

Author: Maksim Sabianin (maksimsab)

Changes

This patch resolves post-commit failure after merging #131347.


Full diff: https://github.com/llvm/llvm-project/pull/152146.diff

1 Files Affected:

  • (modified) llvm/lib/Transforms/Utils/SplitModuleByCategory.cpp (+1-1)
diff --git a/llvm/lib/Transforms/Utils/SplitModuleByCategory.cpp b/llvm/lib/Transforms/Utils/SplitModuleByCategory.cpp
index e75c373e80559..6b18ecee98ec5 100644
--- a/llvm/lib/Transforms/Utils/SplitModuleByCategory.cpp
+++ b/llvm/lib/Transforms/Utils/SplitModuleByCategory.cpp
@@ -318,6 +318,6 @@ void llvm::splitModuleTransitiveFromEntryPoints(
   ModuleSplitter Splitter(std::move(M), std::move(Groups));
   while (Splitter.hasMoreSplits()) {
     ModuleDesc MD = Splitter.getNextSplit();
-    Callback(std::move(MD.releaseModule()));
+    Callback(MD.releaseModule());
   }
 }

@maksimsab
Copy link
Copy Markdown
Contributor Author

Failed job of #131347 could be found here: https://lab.llvm.org/buildbot/#/builders/51/builds/21030

@maksimsab maksimsab closed this Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants