Page MenuHomePhabricator

[Sema] Stable sort OverloadCandidates to remove non-deterministic ordering

Authored by mgrang on Nov 11 2017, 10:36 PM.

Diff Detail


Event Timeline

mgrang created this revision.Nov 11 2017, 10:36 PM
rjmccall accepted this revision.Nov 12 2017, 11:00 AM

Hmm. It looks like the intent is for CompareOverloadCandidatesForDisplay to be a total order, but I'm sure there are cases where it isn't. Okay, LGTM.

This revision is now accepted and ready to land.Nov 12 2017, 11:00 AM
This revision was automatically updated to reflect the committed changes.