-
-
Notifications
You must be signed in to change notification settings - Fork 31.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
gh-132132: Upgrade LLVM on tail calling CI #132098
gh-132132: Upgrade LLVM on tail calling CI #132098
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.
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 whether
eval "$(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.
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.