This is an archive of the discontinued LLVM Phabricator instance.

[flang] Add missing include for std::min
ClosedPublic

Authored by tskeith on Jul 6 2020, 9:19 AM.

Details

Summary

This was causing the build to fail on macos.

Diff Detail

Event Timeline

tskeith created this revision.Jul 6 2020, 9:19 AM
Herald added a project: Restricted Project. · View Herald Transcript
DavidTruby accepted this revision.Jul 6 2020, 9:21 AM

We must have overlapped here somehow, I actually created a revision for this a couple of hours ago
https://reviews.llvm.org/D83227

I'm happy to LGTM this as it is identical to my patch

This revision is now accepted and ready to land.Jul 6 2020, 9:21 AM
klausler accepted this revision.Jul 6 2020, 12:58 PM
This revision was automatically updated to reflect the committed changes.

We must have overlapped here somehow, I actually created a revision for this a couple of hours ago
https://reviews.llvm.org/D83227

Thanks. Phabricator seems very erratic with email. I only received yours a few minutes ago.