[SPIRV] Move instead of copy. (#156637)

maarquitos14 · web-flow · commit b5a1747c8fec · 2025-09-08T14:29:30.000+02:00
diff --git a/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp b/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp
@@ -124,7 +124,7 @@ getSymbolicOperandRequirements(SPIRV::OperandCategory::OperandCategory Category,
       })) {
     return {true,
             {},
-            ReqExts,
+            std::move(ReqExts),
             VersionTuple(),
             VersionTuple()}; // TODO: add versions to extensions.
   }
diff --git a/llvm/lib/Target/SPIRV/SPIRVStructurizer.cpp b/llvm/lib/Target/SPIRV/SPIRVStructurizer.cpp
@@ -43,7 +43,7 @@ using Edge = std::pair<BasicBlock *, BasicBlock *>;
 static void partialOrderVisit(BasicBlock &Start,
                               std::function<bool(BasicBlock *)> Op) {
   PartialOrderingVisitor V(*Start.getParent());
-  V.partialOrderVisit(Start, Op);
+  V.partialOrderVisit(Start, std::move(Op));
 }
 
 // Returns the exact convergence region in the tree defined by `Node` for which

Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ getSymbolicOperandRequirements(SPIRV::OperandCategory::OperandCategory Category,`
`124`	`124`	`})) {`
`125`	`125`	`return {true,`
`126`	`126`	`{},`
`127`		`- ReqExts,`
	`127`	`+ std::move(ReqExts),`
`128`	`128`	`VersionTuple(),`
`129`	`129`	`VersionTuple()}; // TODO: add versions to extensions.`
`130`	`130`	`}`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ using Edge = std::pair<BasicBlock , BasicBlock >;`
`43`	`43`	`static void partialOrderVisit(BasicBlock &Start,`
`44`	`44`	`std::function<bool(BasicBlock *)> Op) {`
`45`	`45`	`PartialOrderingVisitor V(*Start.getParent());`
`46`		`- V.partialOrderVisit(Start, Op);`
	`46`	`+ V.partialOrderVisit(Start, std::move(Op));`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	// Returns the exact convergence region in the tree defined by `Node` for which