Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 33.9k
gh-132132: Upgrade LLVM on tail calling CI#132098
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-132132: Upgrade LLVM on tail calling CI #132098
Uh oh!
There was an error while loading. Please reload this page.
Conversation
Fidget-Spinner commented Apr 4, 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.
Fidget-Spinner commented Apr 4, 2025
@chris-eibl can I take some of your time to request a review please :)? |
chris-eibl commented Apr 4, 2025
Oh no, it's back again - this was already a back and forth (#130040). I think, your fix back then (exclude should stay: clangcl might not understand But then, no will never fire due to clang defines |
chris-eibl commented Apr 4, 2025
Ah - while I was typing you already had the same idea :) |
Python/remote_debugging.c Outdated
| constchar*secname="_PyRuntime"; | ||
| #else | ||
| constchar*secname="PyRuntime"; | ||
| #endif |
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.
Use secname instead of "PyRuntime" below?
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.
Yeah oops 🤦 thanks 3AM things.
| export PATH="/usr/local/opt/llvm/bin:$PATH" | ||
| export PATH="/opt/homebrew/opt/llvm/bin:$PATH" |
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.
wonder whethereval "$(brew shellenv)"
is just enough for both intel and arm mac
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.
That's a good point. Let me try that later.
Fidget-Spinner commented Apr 4, 2025
Well I have absolutely no clue what's going on with Windows here. Will sleep on it and let someone else take over. In the meantime, will just use this PR to upgrade the LLVM version. |
Uh oh!
There was an error while loading. Please reload this page.
chris-eibl commented Apr 5, 2025
@Fidget-Spinner: I've created #132112 to fix |
Fidget-Spinner commented Apr 5, 2025
@chris-eibl can you please send me an email (it's listed on my GH profile), I want to ask some questions. |
chris-eibl commented Apr 5, 2025
Email sent :) |
92fb949 into python:mainUh oh!
There was an error while loading. Please reload this page.
No description provided.