src: Update trace event macros to V8 5.7 version#12127
Closed
Uh oh!
There was an error while loading. Please reload this page.
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
src/tracing/trace_event.h was updated by applying the following changes: 1) Include src/tracing/trace_event_common.h instead of the one from v8 base. 2) Replace all instances of base::Atomic with intptr_t (trace events can only be generated from the main thread for now). 3) Replace instances of V8_INLINE with inline. 4) Eliminate uses of DCHECK. 5) Eliminate uses of V8_UNLIKELY, the branch predictor should be good enough alone. 6) Change the namespace used by trace_event.h from v8::internal::tracing to node::tracing. 7) Remove CallStatsScopedTracer class and related macros (they rely on V8 implementation details). 8) Change ConvertableToTraceFormat to v8::ConvertableToTraceFormat. 9) Add function "static void SetCurrentPlatform(v8::Platform* platform);" to the declaration of TraceEventHelper.
nodejs-github-bot added c++ 
2 tasks
Author
matthewloring commented Mar 29, 2017
Contributor
joshgav commented Mar 29, 2017
haven't reviewed the code yet really, but can confirm it does track AsyncWrap objects with joshgav@fda406f. Thanks @matthewloring! |
Author
matthewloring commented Apr 3, 2017
@nodejs/v8 to review |
Author
matthewloring commented Apr 5, 2017
Author
matthewloring commented Apr 5, 2017
CI is green |
Author
matthewloring commented Apr 5, 2017
Landed in ebeee85 |
3 tasks
hferreiro pushed a commit to brave/node that referenced this pull request Sep 27, 2017
src/tracing/trace_event.h was updated by applying the following changes: 1) Include src/tracing/trace_event_common.h instead of the one from v8 base. 2) Replace all instances of base::Atomic with intptr_t (trace events can only be generated from the main thread for now). 3) Replace instances of V8_INLINE with inline. 4) Eliminate uses of DCHECK. 5) Eliminate uses of V8_UNLIKELY, the branch predictor should be good enough alone. 6) Change the namespace used by trace_event.h from v8::internal::tracing to node::tracing. 7) Remove CallStatsScopedTracer class and related macros (they rely on V8 implementation details). 8) Change ConvertableToTraceFormat to v8::ConvertableToTraceFormat. 9) Add function "static void SetCurrentPlatform(v8::Platform* platform);" to the declaration of TraceEventHelper. PR-URL: nodejs/node#12127 Reviewed-By: Ali Ijaz Sheikh <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment
5 participants
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.
src/tracing/trace_event.h was updated by applying the following changes:
Include src/tracing/trace_event_common.h instead of the one from v8
base.
Replace all instances of base::Atomic with intptr_t (trace events can
only be generated from the main thread for now).
Replace instances of V8_INLINE with inline.
Eliminate uses of DCHECK.
Eliminate uses of V8_UNLIKELY, the branch predictor should be good
enough alone.
Change the namespace used by trace_event.h from v8::internal::tracing
to node::tracing.
Remove CallStatsScopedTracer class and related macros (they rely on
V8 implementation details).
Change ConvertableToTraceFormat to v8::ConvertableToTraceFormat.
Add function "static void SetCurrentPlatform(v8::Platform*
platform);" to the declaration of TraceEventHelper.
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passesAffected core subsystem(s)
src, tracing