This is an archive of the discontinued LLVM Phabricator instance.

[ADT] Don't include <algorithm> in iterator.h
ClosedPublic

Authored by thakis on Apr 16 2021, 9:12 AM.

Details

Summary

As far as I can tell, nothing in iterator.h uses anything from <algorithm>.

Diff Detail

Event Timeline

thakis created this revision.Apr 16 2021, 9:12 AM
thakis requested review of this revision.Apr 16 2021, 9:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 16 2021, 9:12 AM
hans accepted this revision.Apr 16 2021, 9:15 AM
This revision is now accepted and ready to land.Apr 16 2021, 9:15 AM
This revision was landed with ongoing or failed builds.Apr 16 2021, 9:21 AM
This revision was automatically updated to reflect the committed changes.

Perhaps somebody confused <utility> std::move with <algorithm> std::move.

LGTM