gh-132132: Upgrade LLVM on tail calling CI#132098
gh-132132: Upgrade LLVM on tail calling CI#132098Fidget-Spinner merged 17 commits intopython:mainfrom
Conversation
|
@chris-eibl can I take some of your time to request a review please :)? |
|
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 |
|
Ah - while I was typing you already had the same idea :) |
Python/remote_debugging.c
Outdated
| const char* secname = "_PyRuntime"; | ||
| #else | ||
| const char* secname = "PyRuntime"; | ||
| #endif |
There was a problem hiding this comment.
Use secname instead of "PyRuntime" below?
There was a problem hiding this comment.
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.
wonder whether
eval "$(brew shellenv)"
is just enough for both intel and arm mac
There was a problem hiding this comment.
That's a good point. Let me try that later.
|
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. |
|
@Fidget-Spinner: I've created #132112 to fix |
|
@chris-eibl can you please send me an email (it's listed on my GH profile), I want to ask some questions. |
|
Email sent :) |
No description provided.