This is an archive of the discontinued LLVM Phabricator instance.

[lld] Require C++14 in LLD standalone build
ClosedPublic

Authored by int3 on Feb 21 2022, 11:28 AM.

Details

Summary

This is what the Clang standalone build does too. And setting this
seems to be required to get the standalone build to work on my Mac.

Diff Detail

Event Timeline

int3 created this revision.Feb 21 2022, 11:28 AM
int3 requested review of this revision.Feb 21 2022, 11:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 21 2022, 11:28 AM
int3 retitled this revision from [lld-macho] Require C++14 in LLD standalone build to [lld] Require C++14 in LLD standalone build.Feb 21 2022, 6:32 PM
smeenai accepted this revision.Feb 22 2022, 9:39 AM
smeenai added a subscriber: smeenai.

LGTM

This revision is now accepted and ready to land.Feb 22 2022, 9:39 AM
MaskRay accepted this revision.Feb 22 2022, 10:27 AM

I wanted to check this is syntactically identical to Clang's, so we had a clear pattern going forward.

It is indeed! Thanks so much!

Ericson2314 accepted this revision.Feb 22 2022, 12:23 PM
This revision was automatically updated to reflect the committed changes.