[llvm] [TOOLS][UTILS] Added const reference for params with size >= 16 bytes (PR #125082)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 20:57:03 PST 2025


================
@@ -183,7 +183,7 @@ class BugDriver {
   /// MayModifySemantics argument is true, then the cleanups is allowed to
   /// modify how the code behaves.
   ///
-  std::unique_ptr<Module> performFinalCleanups(std::unique_ptr<Module> M,
+  std::unique_ptr<Module> performFinalCleanups(const std::unique_ptr<Module> &M,
----------------
arsenm wrote:

Passing unique_ptr by value or const reference are both suspect. I would expect this to take const Module&

https://github.com/llvm/llvm-project/pull/125082


More information about the llvm-commits mailing list