Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-90815: Define WITH_MIMALLOC macro in PC/pyconfig.h#111528

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
vstinner wants to merge2 commits intopython:mainfromvstinner:mimalloc_win

Conversation

vstinner
Copy link
Member

@vstinnervstinner commentedOct 30, 2023•
edited by bedevere-appbot
Loading

@vstinner
Copy link
MemberAuthor

Example of arecent PR,test.pythoninfo of Windows x64 says:

build.WITH_MIMALLOC: No

@vstinner
Copy link
MemberAuthor

The build fails on Windows. Example:

obmalloc.obj : error LNK2005: _mi_free already defined in alloc.obj [D:\a\cpython\cpython\PCbuild\_freeze_module.vcxproj]

cc@DinoV

@DinoV
Copy link
Contributor

The build fails on Windows. Example:

obmalloc.obj : error LNK2005: _mi_free already defined in alloc.obj [D:\a\cpython\cpython\PCbuild\_freeze_module.vcxproj]

cc@DinoV

I think the issue here is that we're including the files in the Windows build but they're now included via obmalloc.c (this had changed along the way). I'll let the build bots see if#111532 fixes it.

vstinner and stonebig reacted with heart emoji

@vstinner
Copy link
MemberAuthor

@DinoV wrote the correct fix:#111532

@vstinnervstinner deleted the mimalloc_win branchOctober 31, 2023 19:51
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@vstinner@DinoV
[8]ページ先頭
©2009-2025 Movatter.jp