Skip to content
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.

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

Add loose compilation time test #47716

Merged
merged 1 commit into from
Nov 27, 2022
Merged

Add loose compilation time test #47716

merged 1 commit into from
Nov 27, 2022

Conversation

pchintalapudi
Copy link
Member

Follow up test from #47711

This doesn't guarantee that we reliably hit the bug in #47710, since that is the result of an uninitialized variable, but it should at least fail more often when we get junk data.

@brenhinkeller brenhinkeller added the test This change adds or pertains to unit tests label Nov 26, 2022
@brenhinkeller
Copy link
Contributor

Huh, so looks like x86_64-w64-mingw32 is still failing?

@pchintalapudi
Copy link
Member Author

The win64 failure looks more like an out-of-memory error given that it's signaling 10 (which I believe is SIGBUS), perhaps there are zombie processes on that machine eating up memory? The original misc failure doesn't show on any of the testers including win64 though.

@DilumAluthge DilumAluthge merged commit 60668c5 into master Nov 27, 2022
@DilumAluthge DilumAluthge deleted the pc/test-timers branch November 27, 2022 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants