Commit d234475
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
PR-URL: #56238
Refs: #55784
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #58064
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>1 parent a738eb4 commit d234475
2 files changed
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -38,7 +38,7 @@ | |||
| 38 | 38 | ||
| 39 | 39 | # Reset this number to 0 on major V8 upgrades. | |
| 40 | 40 | # Increment by one for each non-official patch applied to deps/v8. | |
| 41 | - 'v8_embedder_string': '-node.2', | ||
| 41 | + 'v8_embedder_string': '-node.3', | ||
| 42 | 42 | ||
| 43 | 43 | ##### V8 defaults for Node.js ##### | |
| 44 | 44 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -592,11 +592,15 @@ path. Add it with -I<path> to the command line | |||
| 592 | 592 | // functions. | |
| 593 | 593 | // Use like: | |
| 594 | 594 | // V8_NOINLINE V8_PRESERVE_MOST void UnlikelyMethod(); | |
| 595 | + #if V8_OS_WIN | ||
| 596 | + # define V8_PRESERVE_MOST | ||
| 597 | + #else | ||
| 595 | 598 | #if V8_HAS_ATTRIBUTE_PRESERVE_MOST | |
| 596 | 599 | # define V8_PRESERVE_MOST __attribute__((preserve_most)) | |
| 597 | 600 | #else | |
| 598 | 601 | # define V8_PRESERVE_MOST /* NOT SUPPORTED */ | |
| 599 | 602 | #endif | |
| 603 | + #endif | ||
| 600 | 604 | ||
| 601 | 605 | ||
| 602 | 606 | // A macro (V8_DEPRECATED) to mark classes or functions as deprecated. | |
0 commit comments