-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Vector3GetHashCodeTest AV'ing on Windows release #20080
Comments
@stephentoub - when I run the tests, this test doesn't seem to run. Has it been disabled? If so, how does one enable it to run the test? |
Yes, I disabled it so that we could update CoreCLR. There's an [ActiveIssue] attribute on the test; just remove it, and it'll start running again. |
@CarolEidt did you have luck getting this to run? |
@danmosemsft Yes,but I haven't got much farther than that. |
OK! let us know if you need anything on our side. |
I was able to repro it and had a possible fix, but after merging with the latest coreclr, it seems to be failing before it hits the previously failing test. I'm continuing to work on it, but may have to pass it to someone else. |
If it helps -- you can run tests just in one folder by going to the |
I have a fix for this issue, and the other problem I was running into, which affects only Checked and Debug versions of the JIT. |
It's great the numeric tests catch these. Do you think https://github.com/dotnet/corefx/issues/15913 is related? |
dotnet/corefx#15913 can't be related since if I interpret the link in there corectly, it's failing on retail too. |
@danmosemsft - I am fixing two issues with dotnet/coreclr#9496. The real issue behind this GetHashCode failure is an issue which causes an AV in retail, and an assert in Checked - though in Checked, if you ran the full set of tests, you would run into the other (Checked/Debug only) AV. So it's possible that dotnet/corefx#15913 is due to the same issue as this. |
With the latest CoreCLR update, this test is consistently AV'ing on Windows.
cc: @mellinoe, @CarolEidt
The text was updated successfully, but these errors were encountered: