Skip to content

Commit 3b04539

Browse files
targosnodejs-github-bot
authored andcommitted
deps: remove problematic comment from v8-internal
GCC emits warnings because of the trailing backslashes. PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent eb58579 commit 3b04539

File tree

2 files changed

+1
-14
lines changed

2 files changed

+1
-14
lines changed

common.gypi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838

3939
# Reset this number to 0 on major V8 upgrades.
4040
# Increment by one for each non-official patch applied to deps/v8.
41-
'v8_embedder_string': '-node.2',
41+
'v8_embedder_string': '-node.3',
4242

4343
##### V8 defaults for Node.js #####
4444

deps/v8/include/v8-internal.h

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -425,19 +425,6 @@ constexpr size_t kMaxCppHeapPointers = 0;
425425
// which all subtypes of a given supertype use contiguous tags. This struct can
426426
// then be used to represent such a type range.
427427
//
428-
// As an example, consider the following type hierarchy:
429-
//
430-
// A F
431-
// / \
432-
// B E
433-
// / \
434-
// C D
435-
//
436-
// A potential type id assignment for range-based type checks is
437-
// {A: 0, B: 1, C: 2, D: 3, E: 4, F: 5}. With that, the type check for type A
438-
// would check for the range [A, E], while the check for B would check range
439-
// [B, D], and for F it would simply check [F, F].
440-
//
441428
// In addition, there is an option for performance tweaks: if the size of the
442429
// type range corresponding to a supertype is a power of two and starts at a
443430
// power of two (e.g. [0x100, 0x13f]), then the compiler can often optimize

0 commit comments

Comments
 (0)