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

Reliance on overflow behavior in tests #467

Closed
jkuszmaul opened this issue Jul 25, 2019 · 3 comments
Closed

Reliance on overflow behavior in tests #467

jkuszmaul opened this issue Jul 25, 2019 · 3 comments
Labels
bug This issue is a bug. p2 This is a standard priority issue

Comments

@jkuszmaul
Copy link

jkuszmaul commented Jul 25, 2019

When I run some of the tests in https://github.com/awslabs/aws-c-common/blob/master/tests/math_test.c (at tag v0.4.2 (018e74c)) optimized using clang to compile them, the tests fail due to the overflow behavior being different from expected. Is there something particular relying on the overflow behavior defined by the tests? I can't tell if the failure of the tests is indicating that I should not be using the compiler/flags that I am, or if it just means that the tests are poorly written.

@jmklix jmklix added bug This issue is a bug. p2 This is a standard priority issue needs-review This issue or pull request needs review from a core team member. labels Aug 28, 2023
@jmklix
Copy link
Member

jmklix commented Dec 13, 2023

Is this still happening in the latest version? If so can you provide more details about the tests that are failing and the errors that you are seeing?

@jmklix jmklix added response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 2 days. and removed needs-review This issue or pull request needs review from a core team member. labels Dec 13, 2023
@jkuszmaul
Copy link
Author

I do not know, and unfortunately the situation where I observed this was at a company which no longer exists and unfortunately I appear to have failed to document exactly what I was observing 4.5 years ago... I do work with a couple of projects that import the SDK right now, so I may prod at this to see if I can surface any issues here.

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 2 days. label Dec 13, 2023
@jmklix
Copy link
Member

jmklix commented Mar 28, 2024

I believe this was fixed in a previous release. If you or anyone else is still sees this happening with any of the tests please open a new issue.

@jmklix jmklix closed this as completed Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

2 participants