This is an archive of the discontinued LLVM Phabricator instance.

[test] std::nth_element regression
AbandonedPublic

Authored by rupprecht on Feb 4 2021, 1:02 PM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

This shows a test case that should pass, but fails, due to a regression in std::nth_element after D93557. The last two elements should be 6 and 7, but they are 7 and 6.

Diff Detail

Event Timeline

rupprecht requested review of this revision.Feb 4 2021, 1:02 PM
rupprecht created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2021, 1:02 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne added a subscriber: ldionne.Feb 5 2021, 7:12 AM

Thanks a lot for finding this regression and reporting it. How did you find it?

I think this can be closed since we're going to go with D96084.

rupprecht abandoned this revision.Feb 5 2021, 8:31 AM

Thanks a lot for finding this regression and reporting it. How did you find it?

We had 10-20 test failures internally that all bisected to this patch, and I logged the before/after of std::nth_element near the test failures. The values here are the relative ordering of the case that failed -- not the actual values, of course :)

I think this can be closed since we're going to go with D96084.