This revision was automatically updated to reflect the committed changes.
Closed by commit rGcf112382ddd0: [ThinLTO] Option to bypass function importing.
(authored by mtrofin).
Changed prior to commit:
https://reviews.llvm.org/D87949?vs=293506&id=293555#toc
Repository:
rG LLVM Github Monorep
tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.
lgtm with comment typo fix
Comment at: llvm/include/llvm/LTO/LTOBackend.h:59
+
+/// Distributed ThinlTO: load the referenced modules, keeping their buffers
+/// alive i
mtrofin updated this revision to Diff 293506.
mtrofin added a comment.
clang tidy
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87949/new/
https://reviews.llvm.org/D87949
Files:
clang/include/clang/CodeGen/BackendUtil.h
clang/lib/CodeGen/Backe
mtrofin updated this revision to Diff 293189.
mtrofin marked 2 inline comments as done.
mtrofin added a comment.
feedback
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87949/new/
https://reviews.llvm.org/D87949
Files:
clang/include/clang/CodeGen
tejohnson added a comment.
Minor comments.
Comment at: clang/test/CodeGen/thinlto_embed_bitcode.ll:17
+; RUN: rm %t1.bc %t2.bc
+; RUN: %clang -target x86_64-unknown-linux-gnu -O2 -o %t-redo.o -x ir
%t-embedded.bc -c -fthinlto-index=%t.o.thinlto.bc -mllvm
-lto-embed-bitcode=po
mtrofin created this revision.
mtrofin added a reviewer: tejohnson.
Herald added subscribers: llvm-commits, cfe-commits, dexonsmith, steven_wu,
hiraditya, inglorion.
Herald added a reviewer: alexshap.
Herald added projects: clang, LLVM.
mtrofin requested review of this revision.
This completes th
