This is an archive of the discontinued LLVM Phabricator instance.

[lld/win] Use C++17 structured bindings
ClosedPublic

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

Details

Summary

No behavior change.

Diff Detail

Event Timeline

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

LGTM - iirc the same was already landed for the macho side.

This revision is now accepted and ready to land.Aug 8 2022, 8:49 AM
thakis added a comment.Aug 8 2022, 8:51 AM

Yes, in D131355. Thanks!

This revision was landed with ongoing or failed builds.Aug 8 2022, 8:53 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 8:53 AM