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

simplify test #1426

Closed
wants to merge 2 commits into from
Closed

simplify test #1426

wants to merge 2 commits into from

Conversation

mattbfb
Copy link
Contributor

@mattbfb mattbfb commented Jun 5, 2024

Differential Revision: D58204748

@facebook-github-bot facebook-github-bot added CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported labels Jun 5, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D58204748

Matt Blagden added 2 commits June 5, 2024 13:02
Summary:

Allow repeated calls to enable the sampling profiler,
updating the thread targeted by the profiler each time.

Previously, the sampling profiler would target only a
single thread. If the runtime was used from another
thread, multiple threads would be trying to use the
runtime in parallel, causing incorrect behavior.

Reviewed By: avp

Differential Revision: D57397716
Summary: Pull Request resolved: #1426

Differential Revision: D58204748
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D58204748

@mattbfb mattbfb closed this Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants