This is an archive of the discontinued LLVM Phabricator instance.

ClangLinkerWrapper: explicitly #include <atomic>
ClosedPublic

Authored by nhaehnle on Jul 26 2022, 7:50 AM.

Details

Summary

This code relied on implicitly having std::atomic available via the
ManagedStatic.h header.

Diff Detail

Event Timeline

nhaehnle created this revision.Jul 26 2022, 7:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 26 2022, 7:50 AM
nhaehnle requested review of this revision.Jul 26 2022, 7:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 26 2022, 7:50 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
efriedma accepted this revision.Jul 26 2022, 9:15 AM

LGTM

You don't really need pre-commit review for a patch that just adds an include, though.

This revision is now accepted and ready to land.Jul 26 2022, 9:15 AM
This revision was landed with ongoing or failed builds.Jul 27 2022, 5:57 AM
This revision was automatically updated to reflect the committed changes.