Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork33.3k
gh-126366: Fix crash if__iter__ raises an exception duringyield from#126369
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
I don't understand how the JIT failures could be related. |
@savannahostrowski@diegorusso My JIT experts, are these known failures, or is there something wrong with my fix? |
savannahostrowski commentedNov 3, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@ZeroIntensity The aarch64 failures are known/expected. This is what we were discussing over Discord a couple of days back. Right now, we are waiting for this to be fixed in the aarch64 macos runners. As for the emulated Linux failures, sometimes we do see failures here, and that's why we maintainthis list of tests to skip. I will note that this test was seen recently passing inhttps://github.com/python/cpython/actions/runs/11647994504/job/32434030552?pr=126339). If this PR has nothing to do with |
Thank you, Savannah! I'll update the skip list and see if that fixes it. |
…pected to the skip list.
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.
LGTM
Uh oh!
There was an error while loading.Please reload this page.
1371295 intopython:mainUh oh!
There was an error while loading.Please reload this page.
Would it be possible to backport the test to 3.12 and 3.13? |
Maybe, but would it be all that useful? Anything that gets backported to those versions will (probably) have to go through the test on main anyway. |
It's useful to check for non-regression: make sure that we don't break this code path in the future. |
Uh oh!
There was an error while loading.Please reload this page.
Apparently,gh-126366 has two issues: one is a thread safety problem inside
list.__init__, and then the other is that theGET_YIELD_FROM_ITERinstruction crashes if a non-native-generator object raises an exception in its__iter__. Since the latter only applies to 3.14, I'm splitting the fix into two PRs.ChainMapof aCounterin threads #126366