This is an archive of the discontinued LLVM Phabricator instance.

Use llvm::sort instead of std::sort where possible
ClosedPublic

Authored by gribozavr on Jul 22 2022, 6:21 PM.

Details

Summary

llvm::sort is beneficial even when we use the iterator-based overload,
since it can optionally shuffle the elements (to detect
non-determinism). However llvm::sort is not usable everywhere, for
example, in compiler-rt.

Diff Detail

Event Timeline

gribozavr created this revision.Jul 22 2022, 6:21 PM
gribozavr requested review of this revision.Jul 22 2022, 6:21 PM
Herald added projects: Restricted Project, Restricted Project, Restricted Project, Restricted Project. · View Herald Transcript
nhaehnle accepted this revision.Jul 23 2022, 5:28 AM
nhaehnle added a subscriber: nhaehnle.

LGTM

This revision is now accepted and ready to land.Jul 23 2022, 5:28 AM
This revision was landed with ongoing or failed builds.Jul 23 2022, 6:19 AM
This revision was automatically updated to reflect the committed changes.