GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||
GeezerVison is showing github.com — gh-132132: Upgrade LLVM on tail calling CI by Fidget-Spinner · Pull Request #132098 · python/cpython Conversation
|
|
@chris-eibl can I take some of your time to request a review please :)? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
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 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Ah - while I was typing you already had the same idea :) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| const char* secname = "_PyRuntime"; | ||
| #else | ||
| const char* secname = "PyRuntime"; | ||
| #endif |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Use secname instead of "PyRuntime" below?
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Yeah oops 🤦 thanks 3AM things.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| export PATH="/usr/local/opt/llvm/bin:$PATH" | ||
| export PATH="/opt/homebrew/opt/llvm/bin:$PATH" |
There was a problem 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
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem 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.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
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. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
@Fidget-Spinner: I've created #132112 to fix |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@chris-eibl can you please send me an email (it's listed on my GH profile), I want to ask some questions. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Email sent :) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.