Move dumpOpcodeStats() to jsi::Instrumentation #53475
Closed
+200
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
meta-cla
bot
added
the
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D81087047 |
lavenzg
force-pushed
the
export-D81087047
branch
from
8741f44 to
2c78c64
Compare
lavenzg
added a commit
to lavenzg/react-native
that referenced
this pull request
Aug 28, 2025
Summary: Rollback Plan: Differential Revision: D81087047
|
This pull request was exported from Phabricator. Differential Revision: D81087047 |
lavenzg
added a commit
to lavenzg/react-native
that referenced
this pull request
Aug 28, 2025
Summary: Rollback Plan: Differential Revision: D81087047
lavenzg
force-pushed
the
export-D81087047
branch
from
2c78c64 to
f0b7934
Compare
|
This pull request was exported from Phabricator. Differential Revision: D81087047 |
lavenzg
force-pushed
the
export-D81087047
branch
from
f0b7934 to
a7ee810
Compare
lavenzg
added a commit
to lavenzg/react-native
that referenced
this pull request
Aug 29, 2025
…acebook#53475) Summary: This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Differential Revision: D81087047
|
This pull request was exported from Phabricator. Differential Revision: D81087047 |
lavenzg
added a commit
to lavenzg/react-native
that referenced
this pull request
Aug 29, 2025
…acebook#53475) Summary: Pull Request resolved: facebook#53475 This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Differential Revision: D81087047
lavenzg
force-pushed
the
export-D81087047
branch
2 times, most recently
from
abfe7eb to
60ec141
Compare
lavenzg
added a commit
to lavenzg/react-native
that referenced
this pull request
Sep 2, 2025
…acebook#53475) Summary: This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Differential Revision: D81087047
|
This pull request was exported from Phabricator. Differential Revision: D81087047 |
lavenzg
added a commit
to lavenzg/react-native
that referenced
this pull request
Sep 2, 2025
…acebook#53475) Summary: Pull Request resolved: facebook#53475 This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Differential Revision: D81087047
lavenzg
force-pushed
the
export-D81087047
branch
from
60ec141 to
300fc56
Compare
Differential Revision: D80740969
Differential Revision: D80747463
…race, [SH] Move dumpSampledTraceToProfile() to IHermes Differential Revision: D81075460
lavenzg
force-pushed
the
export-D81087047
branch
from
300fc56 to
1293cb4
Compare
lavenzg
added a commit
to lavenzg/react-native
that referenced
this pull request
Sep 3, 2025
…acebook#53475) Summary: This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Reviewed By: tsaichien Differential Revision: D81087047
…acebook#53475) Summary: Pull Request resolved: facebook#53475 This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Reviewed By: tsaichien Differential Revision: D81087047
|
This pull request was exported from Phabricator. Differential Revision: D81087047 |
lavenzg
force-pushed
the
export-D81087047
branch
from
1293cb4 to
8bc8fbf
Compare
facebook-github-bot
pushed a commit
to facebook/hermes
that referenced
this pull request
Sep 4, 2025
Summary: X-link: facebook/react-native#53475 This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Changelog: [Internal] Reviewed By: tsaichien Differential Revision: D81087047 fbshipit-source-id: e145aafea7459a161fca04ffc30f0838ee6c03c6
facebook-github-bot
pushed a commit
to facebook/hermes
that referenced
this pull request
Sep 4, 2025
Summary: X-link: facebook/react-native#53475 This is cleanup of IHermesExtra. Move the last method in IHermesExtra, dumpOpcodeStats(), to jsi::Instrumentation, since other profile stats dumping methods live in that interface as well. Changelog: [Internal] Reviewed By: tsaichien Differential Revision: D81087047 fbshipit-source-id: e145aafea7459a161fca04ffc30f0838ee6c03c6
|
This pull request has been merged in 863184f. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Differential Revision: D81087047