-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
also disable is_foldable
test on other 32bit platforms
#54910
also disable is_foldable
test on other 32bit platforms
#54910
Conversation
which of the functions isn't inferring correctly? we probably do want the basic math to infer correctly on 32 but systems |
This is the test Lines 1605 to 1609 in 696d9c3
|
Basically I'm proposing that given the comment that was on this test, we merge this so that we can merge the buildkite change and the issue can be addressed in due course per the comment. |
interesting... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Debugging this kind of issue is very troublesome without easy access to a 32-bit systems, so I think it's okay to disable it for now.
so launching 32 bit Julia (1.10.4) on a 64 bit linux machine doesn't reproduce this... |
Try a nightly? |
grr. Cthulhu is broken on nightly... |
fixes #54910 properly. The recursion heuristic was getting mad at this for some reason...
Is it really broken? Cthulhu@2.12.7 seems to be working fine on master. |
ah I had an outdated version |
Follow-on from #54323
@aviatesk I think this should also be disabled on linux? It's been failing there
On
test i686-linux-gnu