Skip to content

Commit add506a

Browse files
StefanStojanovictargos
authored andcommitted
deps: define V8_PRESERVE_MOST as no-op on Windows
It's causing linker errors with node.lib in node-gyp and potentially breaks other 3rd party tools Refs: #55784 PR-URL: #61898 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]>
1 parent b6a9674 commit add506a

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

common.gypi

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

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

4545
##### V8 defaults for Node.js #####
4646

deps/v8/include/v8config.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -599,11 +599,15 @@ path. Add it with -I<path> to the command line
599599
// functions.
600600
// Use like:
601601
// V8_NOINLINE V8_PRESERVE_MOST void UnlikelyMethod();
602+
#if V8_OS_WIN
603+
# define V8_PRESERVE_MOST
604+
#else
602605
#if V8_HAS_ATTRIBUTE_PRESERVE_MOST
603606
# define V8_PRESERVE_MOST __attribute__((preserve_most))
604607
#else
605608
# define V8_PRESERVE_MOST /* NOT SUPPORTED */
606609
#endif
610+
#endif
607611

608612

609613
// A macro (V8_DEPRECATED) to mark classes or functions as deprecated.

0 commit comments

Comments
 (0)