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-115103: Enable internal mimalloc assertions in debug builds#116343

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

Merged

Conversation

colesbury
Copy link
Contributor

@colesburycolesbury commentedMar 4, 2024
edited by bedevere-appbot
Loading

This setsMI_DEBUG to2 in debug builds to enablemi_assert_internal() calls. Expensive internal assertions are not enabled.

This also disables an assertion in free-threaded builds that would be triggered by the free-threaded GC because we traverse heaps that are not owned by the current thread.

This sets `MI_DEBUG` to `2` in debug builds to enable`mi_assert_internal()` calls. Expensive internal assertions are notenabled.This also disables an assertion in free-threaded builds that would betriggered by the free-threaded GC because we traverse heaps that are notowned by the current thread.
@colesbury
Copy link
ContributorAuthor

!buildbot nogil

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@colesbury for commitd336146 🤖

The command will test the builders whose names match following regular expression:nogil

The builders matched are:

  • x86-64 MacOS Intel ASAN NoGIL PR
  • x86-64 MacOS Intel NoGIL PR
  • ARM64 MacOS M1 Refleaks NoGIL PR
  • ARM64 MacOS M1 NoGIL PR
  • AMD64 Ubuntu NoGIL Refleaks PR
  • AMD64 Ubuntu NoGIL PR
  • AMD64 Windows Server 2022 NoGIL PR

@colesbury
Copy link
ContributorAuthor

The context is that the changes to mimalloc related to QSBR would benefit from the internal assertions being enabled.

Copy link
Contributor

@DinoVDinoV left a comment

Choose a reason for hiding this comment

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

LGTM!

@colesburycolesbury merged commit72714c0 intopython:mainMar 5, 2024
@colesburycolesbury deleted the gh-115103-mimalloc-assertions branchMarch 5, 2024 18:54
adorilson pushed a commit to adorilson/cpython that referenced this pull requestMar 25, 2024
…ython#116343)This sets `MI_DEBUG` to `2` in debug builds to enable `mi_assert_internal()`calls. Expensive internal assertions are not enabled.This also disables an assertion in free-threaded builds that would betriggered by the free-threaded GC because we traverse heaps that are notowned by the current thread.
diegorusso pushed a commit to diegorusso/cpython that referenced this pull requestApr 17, 2024
…ython#116343)This sets `MI_DEBUG` to `2` in debug builds to enable `mi_assert_internal()`calls. Expensive internal assertions are not enabled.This also disables an assertion in free-threaded builds that would betriggered by the free-threaded GC because we traverse heaps that are notowned by the current thread.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@DinoVDinoVDinoV approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@colesbury@bedevere-bot@DinoV

[8]ページ先頭

©2009-2025 Movatter.jp