Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-130039: Tailcall for windows builds #130040

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.

Sign up for GitHub

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

Merged
Fidget-Spinner merged 17 commits into from
Mar 11, 2025
Merged

gh-130039: Tailcall for windows builds #130040

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ab0a611
Tail-calling flag for Window
Fidget-Spinner Feb 12, 2025
d57c175
Update pythoncore.vcxproj
Fidget-Spinner Feb 12, 2025
3eb568f
Add Windows to the tailcall CI
Fidget-Spinner Feb 12, 2025
4dd19cf
Update tail-call.yml
Fidget-Spinner Feb 12, 2025
7f91920
Update tail-call.yml
Fidget-Spinner Feb 12, 2025
c3d656b
Update pycore_debug_offsets.h
Fidget-Spinner Feb 12, 2025
3638cda
Update pycore_debug_offsets.h
Fidget-Spinner Feb 12, 2025
b8b79ed
Update pycore_debug_offsets.h
Fidget-Spinner Feb 12, 2025
d3766d8
Merge branch 'main' into tailcall_for_windows
Fidget-Spinner Feb 12, 2025
c811b8a
Address review by using env vars
Fidget-Spinner Feb 13, 2025
904b7ee
Merge remote-tracking branch 'upstream/main' into tailcall_for_windows
Fidget-Spinner Feb 13, 2025
991b50f
Merge remote-tracking branch 'upstream/main' into tailcall_for_windows
Fidget-Spinner Feb 15, 2025
166187c
Update pythoncore.vcxproj
Fidget-Spinner Feb 15, 2025
5f7ad24
Update .github/workflows/tail-call.yml
Fidget-Spinner Mar 5, 2025
6d7b689
Update tail-call.yml
Fidget-Spinner Mar 5, 2025
aadfec4
Address review
Fidget-Spinner Mar 5, 2025
1d7c051
Partially revert "Address review"
Fidget-Spinner Mar 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update pycore_debug_offsets.h
  • Loading branch information
@Fidget-Spinner
Fidget-Spinner committed Feb 12, 2025
commit b8b79ed9ed0658221d7d2d4099db6f36cc192604
4 changes: 2 additions & 2 deletions Include/internal/pycore_debug_offsets.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ extern "C" {
declaration \
_GENERATE_DEBUG_SECTION_LINUX(name)

#if defined(MS_WINDOWS)
#if defined(MS_WINDOWS) && !defined(__clang__)
Copy link
Member

@zooba zooba Mar 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this need an alternate definition for Windows + clang?

Copy link
Member Author

@Fidget-Spinner Fidget-Spinner Mar 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, below there's a default that it uses is we're not one of the supported platforms. And that default works for Clang Windows.

#define _GENERATE_DEBUG_SECTION_WINDOWS(name) \
_Pragma(Py_STRINGIFY(section(Py_STRINGIFY(name), read, write))) \
__declspec(allocate(Py_STRINGIFY(name)))
Expand All @@ -39,7 +39,7 @@ extern "C" {
#define _GENERATE_DEBUG_SECTION_APPLE(name)
#endif

#if defined(__linux__) && (defined(__GNUC__) || (defined(__clang__) && !defined(MS_WINDOWS)))
#if defined(__linux__) && (defined(__GNUC__) || defined(__clang__))
#define _GENERATE_DEBUG_SECTION_LINUX(name) \
__attribute__((section("." Py_STRINGIFY(name)))) \
__attribute__((used))
Expand Down
Loading