Skip to content
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-90815: Fix test_embed for Windows PGO build with mimalloc #94790

Merged
merged 3 commits into from Jul 13, 2022

Conversation

neonene
Copy link
Contributor

@neonene neonene commented Jul 12, 2022

Fixes the failure of PGO building with mimalloc on Windows, ensuring that test_bpo20891 does not break profiling data (python31*.pgc).

Automerge-Triggered-By: GH:tiran

@neonene
Copy link
Contributor Author

@neonene neonene commented Jul 12, 2022

@ericsnowcurrently Could you take a look at this patch, please?

@tiran
Copy link
Member

@tiran tiran commented Jul 13, 2022

@neonene This looks like a general fix that should be backported to 3.11 and 3.10, too. Do you agree?

tiran
tiran approved these changes Jul 13, 2022
@neonene
Copy link
Contributor Author

@neonene neonene commented Jul 13, 2022

I agree. This patch would be better for profiling on 3.11 and 3.10.

@tiran tiran added needs backport to 3.10 needs backport to 3.11 🤖 automerge labels Jul 13, 2022
@miss-islington miss-islington merged commit 4a6bb30 into python:main Jul 13, 2022
12 checks passed
@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jul 13, 2022

Thanks @neonene for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒🤖

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Jul 13, 2022

GH-94805 is a backport of this pull request to the 3.11 branch.

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Jul 13, 2022

GH-94806 is a backport of this pull request to the 3.10 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 13, 2022
…ythonGH-94790)

Fixes the failure of PGO building with `mimalloc` on Windows, ensuring that `test_bpo20891` does not break profiling data (`python31*.pgc`).
(cherry picked from commit 4a6bb30)

Co-authored-by: neonene <53406459+neonene@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 13, 2022
…ythonGH-94790)

Fixes the failure of PGO building with `mimalloc` on Windows, ensuring that `test_bpo20891` does not break profiling data (`python31*.pgc`).
(cherry picked from commit 4a6bb30)

Co-authored-by: neonene <53406459+neonene@users.noreply.github.com>
miss-islington added a commit that referenced this issue Jul 13, 2022
Fixes the failure of PGO building with `mimalloc` on Windows, ensuring that `test_bpo20891` does not break profiling data (`python31*.pgc`).
(cherry picked from commit 4a6bb30)

Co-authored-by: neonene <53406459+neonene@users.noreply.github.com>
miss-islington added a commit that referenced this issue Jul 13, 2022
Fixes the failure of PGO building with `mimalloc` on Windows, ensuring that `test_bpo20891` does not break profiling data (`python31*.pgc`).
(cherry picked from commit 4a6bb30)

Co-authored-by: neonene <53406459+neonene@users.noreply.github.com>
@neonene neonene deleted the pgcgen branch Jul 13, 2022
iritkatriel pushed a commit to iritkatriel/cpython that referenced this issue Jul 19, 2022
…ythonGH-94790)

Fixes the failure of PGO building with `mimalloc` on Windows, ensuring that `test_bpo20891` does not break profiling data (`python31*.pgc`).
@ericsnowcurrently
Copy link
Member

@ericsnowcurrently ericsnowcurrently commented Jul 21, 2022

(FTR: gh-65090)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 automerge skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants