[PATCH] D57115: [llvm] Remove dependency on <algorithm> [NFC]

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 24 11:12:03 PST 2019


dblaikie added a comment.

In D57115#1369756 <https://reviews.llvm.org/D57115#1369756>, @mgrang wrote:

> I wanted to clean up the inclusions of algorithm due to std::sort. But seems like this change may potentially break some things. So for now we can let it pass? (Meaning I can abandon this patch).


Yep - thanks for trying, but unless you can get something like IWYU working to help ensure this cleanup is a bit more robust - I'd probably suggest just abandoning this change, unfortunately.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57115/new/

https://reviews.llvm.org/D57115





More information about the llvm-commits mailing list