This is an archive of the discontinued LLVM Phabricator instance.

[lld/win] Use C++17 nested namespace syntax in most places
ClosedPublic

Authored by thakis on Aug 8 2022, 8:35 AM.

Details

Summary

Like D131354, but for COFF.

No behavior change.

Diff Detail

Event Timeline

thakis created this revision.Aug 8 2022, 8:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 8:35 AM
thakis requested review of this revision.Aug 8 2022, 8:35 AM
mstorsjo accepted this revision.Aug 8 2022, 8:45 AM

Looks reasonable to me.

I guess the coding style guidelines will be updated for C++17 features eventually (about what features to use and not use) - this particular change seems uncontroversial in any case.

This revision is now accepted and ready to land.Aug 8 2022, 8:45 AM
This revision was landed with ongoing or failed builds.Aug 8 2022, 8:48 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 8:48 AM
sbc100 added a subscriber: sbc100.Mar 6 2023, 9:41 AM

In general, when making these kind of generic changes, would it be possible to also include wasm-ld (lld/wasm) so that we keep things as in-sync as possible?