Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

gh-90815: Exclude mimalloc .c files from Windows build #111532

Merged
merged 2 commits into from
Oct 31, 2023

Conversation

DinoV
Copy link
Contributor

@DinoV DinoV commented Oct 31, 2023

Remove mimalloc .c files from Windows build

@vstinner
Copy link
Member

There are some compiler warnings: 'fprintf' : format string '%ld' requires an argument of type 'long', but variadic argument 1 has type 'unsigned __int64' [D:\a\cpython\cpython\PCbuild\pythoncore.vcxproj]

@vstinner
Copy link
Member

Tests pass!

@corona10 corona10 changed the title Don't include mimalloc .c's in Windows build gh-90815: Exclude mimalloc .c files from Windows build Oct 31, 2023
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM. Objects/obmalloc.c uses #include "pycore_mimalloc.h", so already includes mimalloc code "statically".

@vstinner
Copy link
Member

I don't click on "Enable auto-merge", since I would like to see if the warnings are gone ;-)

DinoV added 2 commits October 31, 2023 10:23
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:
@DinoV DinoV force-pushed the nogil/mimalloc_windows branch from 846b2af to 7894cca Compare October 31, 2023 17:24
@DinoV DinoV merged commit c42347d into python:main Oct 31, 2023
FullteaR pushed a commit to FullteaR/cpython that referenced this pull request Nov 3, 2023
…111532)

* Don't include mimalloc .c's in Windows build
* Fix warnings on Windows related to mimalloc
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…111532)

* Don't include mimalloc .c's in Windows build
* Fix warnings on Windows related to mimalloc
@DinoV DinoV deleted the nogil/mimalloc_windows branch May 31, 2024 18:23
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…111532)

* Don't include mimalloc .c's in Windows build
* Fix warnings on Windows related to mimalloc
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants