Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34k
GH-137218: Update make for JIT stencils#137265
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
GH-137218: Update make for JIT stencils #137265
Uh oh!
There was an error while loading. Please reload this page.
Conversation
savannahostrowski commented Jul 31, 2025 • 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.
savannahostrowski commented Jul 31, 2025 • 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.
Fixes the segfault on my machine but feel free to test if we think there's anything dubious here. I opted to remove the autoconf substitution here since this seems pretty straightforward. |
brandtbucher commented Jul 31, 2025
Could we do Also, does the PCbuild logic need to be updated? Or is it good? |
eec5f75 to ba48cc7Comparesavannahostrowski commented Aug 1, 2025
I'm no That said...I...I am truly perplexed as I can no longer reproduce the segfault. The first commit from last night fixed the issue locally, so I know that this did solve the issue I was seeing, but...???? @brandtbucher Can you still repro? |
Uh oh!
There was an error while loading. Please reload this page.
savannahostrowski commented Aug 2, 2025
I know what the issue is here, will fix later. |
devdanzin commented Aug 4, 2025
I have the exact same problem, it suddenly stopped crashing and I haven't figured out a way to get into the segfaulting state ever since. So it seems normal for this abnormal issue. |
85c1ef6 into python:mainUh oh!
There was an error while loading. Please reload this page.
bedevere-bot commented Sep 19, 2025
|
vstinner commented Sep 19, 2025
This change broke the build on FreeBSD: https://buildbot.python.org/#/builders/1223/builds/7107 |
savannahostrowski commented Sep 19, 2025
Ah, sorry about that. CI didn't catch it. I understand what happened here. I'll open a PR to fix it. |
This PR normalizes stencil names and then modifies the
Makefileto select the appropriate JIT stencil header based on the host._pyreplwith low value forJUMP_BACKWARD_INITIAL_VALUE#137218