Skip to content

feature/spaceship: Merge fix for test timing assumption #1212

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 10 commits into from
Aug 20, 2020
Merged

feature/spaceship: Merge fix for test timing assumption #1212

merged 10 commits into from
Aug 20, 2020

Conversation

@StephanTLavavej
Copy link
Member

@StephanTLavavej StephanTLavavej commented Aug 20, 2020 •
edited
Loading

Merge master into feature/spaceship to pick up #1193 and prevent sporadic test failures.

⚠️ Note to self: Commit this as a merge, not a squash!

AlexGuteniev and others added 10 commits August 12, 2020 18:10
@AlexGuteniev @BillyONeal
@AdamBucior @StephanTLavavej
Implement atomic_ref (#2, p0019r8) (#843)
f3ca37d 
Co-authored-by: Billy O'Neal <[email protected]>
Co-authored-by: Adam Bucior <[email protected]>
Co-authored-by: Stephan T. Lavavej <[email protected]>
@BillyONeal
Guard noexcept annotated function pointer type for __cpp_noexcept_fun…
87dc1d3 
…ction_type. (#1150)

Resolves VSO-1166234.
@miscco @cpplearner
@StephanTLavavej @CaseyCarter
Implement counted_iterator (#1083)
1e42166 
Co-authored-by: S. B. Tam <[email protected]>
Co-authored-by: Stephan T. Lavavej <[email protected]>
Co-authored-by: Casey Carter <[email protected]>
@CaseyCarter
/std:c++latest configs are /permissive or /permissive- (#1162)
02bf0cf 
So our test coverage won't be damaged when the compiler changes `/std:c++latest` to imply `/permissive-`. I do so by adding `/permissive-` to every `/std:c++latest` command line that doesn't yet have it, and then reverting one configuration in each matrix to `/permissive`. It seems sensible to me that the majority of our coverage should change from permissive to strict mode to reflect the change in compiler default.
@AlexGuteniev
test_atomic_wait.hpp: remove another timing assumption (#1193)
e7451b3
@statementreply
<charconv>: fix from_chars("0.fffffffffffff8p-1022") (#934)
9fb6caa 
Fixes #931.
@AlexGuteniev @StephanTLavavej @CaseyCarter
Test _Countl_zero_bsr with a dedicated test (#1166)
3de20f9 
Resolves #1120.

Co-authored-by: Stephan T. Lavavej <[email protected]>
Co-authored-by: Casey Carter <[email protected]>
@CaseyCarter
Correct issues from investigation of VSO-938163 (#1191)
1306963
@AlexGuteniev @StephanTLavavej
Use SRWLOCK in atomic_load operation on shared_ptr (#1200)
a68edbe 
Fixes #370.

Co-authored-by: Stephan T. Lavavej <[email protected]>
@StephanTLavavej
Update __cpp_lib_coroutine. (#1204)
3cca473
@StephanTLavavej StephanTLavavej added test Related to test code uncharted Excluded from the Status Chart labels Aug 20, 2020
@StephanTLavavej StephanTLavavej requested a review from a team as a code owner August 20, 2020 00:50
@StephanTLavavej StephanTLavavej merged commit b9642bf into feature/spaceship Aug 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@CaseyCarter CaseyCarter CaseyCarter approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

test Related to test code uncharted Excluded from the Status Chart

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@StephanTLavavej @CaseyCarter @AlexGuteniev @BillyONeal @miscco @statementreply