[PATCH] D140627: [orc] Pass big parameters by reference, instead of by value

Feng Zou via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 25 20:29:57 PST 2022


fzou1 updated this revision to Diff 485266.
fzou1 added a comment.

Apply clang format


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140627/new/

https://reviews.llvm.org/D140627

Files:
  llvm/tools/lli/lli.cpp


Index: llvm/tools/lli/lli.cpp
===================================================================
--- llvm/tools/lli/lli.cpp
+++ llvm/tools/lli/lli.cpp
@@ -897,19 +897,19 @@
     CacheManager = std::make_unique<LLIObjectCache>(ObjectCacheDir);
 
     Builder.setCompileFunctionCreator(
-      [&](orc::JITTargetMachineBuilder &JTMB)
+        [&](orc::JITTargetMachineBuilder &JTMB)
             -> Expected<std::unique_ptr<orc::IRCompileLayer::IRCompiler>> {
-        if (LazyJITCompileThreads > 0)
-          return std::make_unique<orc::ConcurrentIRCompiler>(std::move(JTMB),
-                                                        CacheManager.get());
+          if (LazyJITCompileThreads > 0)
+            return std::make_unique<orc::ConcurrentIRCompiler>(
+                std::move(JTMB), CacheManager.get());
 
-        auto TM = JTMB.createTargetMachine();
-        if (!TM)
-          return TM.takeError();
+          auto TM = JTMB.createTargetMachine();
+          if (!TM)
+            return TM.takeError();
 
-        return std::make_unique<orc::TMOwningSimpleCompiler>(std::move(*TM),
-                                                        CacheManager.get());
-      });
+          return std::make_unique<orc::TMOwningSimpleCompiler>(
+              std::move(*TM), CacheManager.get());
+        });
   }
 
   // Set up LLJIT platform.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D140627.485266.patch
Type: text/x-patch
Size: 1359 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221226/183a3c29/attachment.bin>


More information about the llvm-commits mailing list