This is an archive of the discontinued LLVM Phabricator instance.

[clang] Change std::sort to llvm::sort in response to r327219
ClosedPublic

Authored by mgrang on Mar 10 2018, 7:50 PM.

Details

Reviewers
rsmith
rnk
Summary

r327219 added wrappers to std::sort which randomly shuffle the container before sorting.
This will help in uncovering non-determinism caused due to undefined sorting
order of objects having the same key.

To make use of that infrastructure we need to invoke llvm::sort instead of std::sort.

Diff Detail

Repository
rC Clang

Event Timeline

mgrang created this revision.Mar 10 2018, 7:50 PM
rsmith added inline comments.Mar 10 2018, 10:10 PM
include/clang-c/Index.h
2133 ↗(On Diff #137938)

This shouldn't be changed.

lib/AST/VTableBuilder.cpp
2109

Reindent this (and other calls) please.

mgrang updated this revision to Diff 137966.Mar 11 2018, 6:44 PM

Fixed indentation.

Ping for reviews please.

rnk accepted this revision.Mar 26 2018, 2:24 PM

lgtm

This revision is now accepted and ready to land.Mar 26 2018, 2:24 PM
mgrang closed this revision.Mar 29 2018, 12:09 PM