[PATCH] D130574: ClangLinkerWrapper: explicitly #include

2022-07-27 Thread Nicolai Hähnle via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3e874bcf0642: ClangLinkerWrapper: explicitly #include (authored by nhaehnle). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D130574: ClangLinkerWrapper: explicitly #include

2022-07-26 Thread Eli Friedman via Phabricator via cfe-commits
efriedma accepted this revision. efriedma added a comment. This revision is now accepted and ready to land. LGTM You don't really need pre-commit review for a patch that just adds an include, though. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1

[PATCH] D130574: ClangLinkerWrapper: explicitly #include

2022-07-26 Thread Nicolai Hähnle via Phabricator via cfe-commits
nhaehnle created this revision. nhaehnle added reviewers: efriedma, lattner. Herald added a project: All. nhaehnle requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This code relied on implicitly having std::atomic available via the ManagedSt