This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGc732c3df749b: [OpenMP] Initial Implementation of LTO and
bitcode linking in linker wrapper (authored by jhuber6).
Changed prior to commit:
jhuber6 added inline comments.
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:8154
+for (auto TI = OpenMPTCRange.first, TE = OpenMPTCRange.second; TI != TE;
+ ++TI) {
+ const ToolChain *TC = TI->second;
jdoerfert wrote:
> Nit: maybe `for
jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.
LG, as part of the patch set, and given the runtime test for coverage.
Some nits.
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:8154
+for (auto TI =
jhuber6 updated this revision to Diff 404528.
jhuber6 added a comment.
Moving adding OpenMPOpt to LTO pipeline to a new patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116975/new/
https://reviews.llvm.org/D116975
Files:
jhuber6 updated this revision to Diff 398821.
jhuber6 added a comment.
Herald added subscribers: ormris, steven_wu.
Fix tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116975/new/
https://reviews.llvm.org/D116975
Files:
jhuber6 updated this revision to Diff 398748.
jhuber6 added a comment.
Undoing accidental clang format.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116975/new/
https://reviews.llvm.org/D116975
Files:
clang/lib/Driver/Driver.cpp
jhuber6 created this revision.
jhuber6 added reviewers: jdoerfert, gregrodgers, JonChesterfield, ronlieb.
Herald added subscribers: guansong, hiraditya, inglorion, yaxunl, mgorny.
jhuber6 requested review of this revision.
Herald added subscribers: llvm-commits, cfe-commits, sstefan1.
Herald added