Visitar URL original
deps: patch V8 to support compilation with MSVC · nodejs/node@b9d806a · GitHub
Skip to content

Commit b9d806a

Browse files
StefanStojanovictargos
StefanStojanovic
authored and
targos
committed
deps: patch V8 to support compilation with MSVC
This patches V8 v12.2 for Windows, by fixing multiple compilation errors caused by V8 being a Clang-oriented project. There are various types of errors fixed by this going from changing `using` directives and renaming to overcoming the differences in which Clang and MSVC see templates and metaprogramming. The changes introduced here are strictly meant as a patch only, so they shouldn't be pushed upstream. Refs: targos#13 Refs: targos#14 PR-URL: #51362 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
1 parent 63b58bc commit b9d806a

31 files changed

Lines changed: 129 additions & 131 deletions

‎common.gypi‎

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

3838
# Reset this number to 0 on major V8 upgrades.
3939
# Increment by one for each non-official patch applied to deps/v8.
40-
'v8_embedder_string': '-node.6',
40+
'v8_embedder_string': '-node.7',
4141

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

‎deps/v8/src/builtins/builtins-collections-gen.cc‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2782,10 +2782,9 @@ TNode<Word32T> WeakCollectionsBuiltinsAssembler::ShouldShrink(
27822782

27832783
TNode<IntPtrT> WeakCollectionsBuiltinsAssembler::ValueIndexFromKeyIndex(
27842784
TNode<IntPtrT> key_index) {
2785-
return IntPtrAdd(
2786-
key_index,
2787-
IntPtrConstant(EphemeronHashTable::TodoShape::kEntryValueIndex -
2788-
EphemeronHashTable::kEntryKeyIndex));
2785+
return IntPtrAdd(key_index,
2786+
IntPtrConstant(EphemeronHashTable::ShapeT::kEntryValueIndex -
2787+
EphemeronHashTable::kEntryKeyIndex));
27892788
}
27902789

27912790
TF_BUILTIN(WeakMapConstructor, WeakCollectionsBuiltinsAssembler) {

‎deps/v8/src/codegen/code-stub-assembler.cc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9455,7 +9455,7 @@ void CodeStubAssembler::NameDictionaryLookup(
94559455
CAST(UnsafeLoadFixedArrayElement(dictionary, index));
94569456
GotoIf(TaggedEqual(current, undefined), if_not_found);
94579457
if (mode == kFindExisting) {
9458-
if (Dictionary::TodoShape::kMatchNeedsHoleCheck) {
9458+
if (Dictionary::ShapeT::kMatchNeedsHoleCheck) {
94599459
GotoIf(TaggedEqual(current, TheHoleConstant()), &next_probe);
94609460
}
94619461
current = LoadName<Dictionary>(current);

‎deps/v8/src/compiler/turboshaft/assembler.h‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3934,8 +3934,14 @@ class TSAssembler
39343934
: public Assembler<reducer_list<TurboshaftAssemblerOpInterface, Reducers...,
39353935
TSReducerBase>> {
39363936
public:
3937+
#ifdef _WIN32
3938+
explicit TSAssembler(Graph& input_graph, Graph& output_graph,
3939+
Zone* phase_zone)
3940+
: Assembler(input_graph, output_graph, phase_zone) {}
3941+
#else
39373942
using Assembler<reducer_list<TurboshaftAssemblerOpInterface, Reducers...,
39383943
TSReducerBase>>::Assembler;
3944+
#endif
39393945
};
39403946

39413947
#include "src/compiler/turboshaft/undef-assembler-macros.inc"

‎deps/v8/src/compiler/turboshaft/code-elimination-and-simplification-phase.cc‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,8 @@ void CodeEliminationAndSimplificationPhase::Run(Zone* temp_zone) {
3232
// (which, for simplificy, doesn't use the Assembler helper
3333
// methods, but only calls Next::ReduceLoad/Store).
3434
DuplicationOptimizationReducer,
35-
ValueNumberingReducer>::Run(temp_zone);
35+
VariableReducerHotfix,
36+
ValueNumberingReducer>::Run<false>(temp_zone);
3637
}
3738

3839
} // namespace v8::internal::compiler::turboshaft

‎deps/v8/src/compiler/turboshaft/copying-phase.h‎

Lines changed: 25 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,18 @@ struct PaddingSpace {
3636
V8_EXPORT_PRIVATE std::ostream& operator<<(std::ostream& os,
3737
PaddingSpace padding);
3838

39+
template <class Next>
40+
class VariableReducerHotfix : public Next {
41+
public:
42+
TURBOSHAFT_REDUCER_BOILERPLATE()
43+
44+
void SetVariable(Variable var, OpIndex new_index) {}
45+
Variable NewLoopInvariantVariable(MaybeRegisterRepresentation rep) { return Variable(); }
46+
47+
OpIndex GetVariable(Variable var) { return OpIndex(); }
48+
OpIndex GetPredecessorValue(Variable var, int predecessor_index) { return OpIndex(); }
49+
};
50+
3951
template <typename Next>
4052
class ReducerBaseForwarder;
4153
template <typename Next>
@@ -46,6 +58,9 @@ class GraphVisitor : public Next {
4658
template <typename N>
4759
friend class ReducerBaseForwarder;
4860

61+
private:
62+
bool contains_variable_reducer_;
63+
4964
public:
5065
TURBOSHAFT_REDUCER_BOILERPLATE()
5166

@@ -66,7 +81,8 @@ class GraphVisitor : public Next {
6681
// `trace_reduction` is a template parameter to avoid paying for tracing at
6782
// runtime.
6883
template <bool trace_reduction>
69-
void VisitGraph() {
84+
void VisitGraph(bool contains_variable_reducer) {
85+
contains_variable_reducer_ = contains_variable_reducer;
7086
Asm().Analyze();
7187

7288
// Creating initial old-to-new Block mapping.
@@ -177,8 +193,7 @@ class GraphVisitor : public Next {
177193
DCHECK(old_index.valid());
178194
OpIndex result = op_mapping_[old_index];
179195

180-
if constexpr (reducer_list_contains<typename Next::ReducerList,
181-
VariableReducer>::value) {
196+
if (contains_variable_reducer_) {
182197
if (!result.valid()) {
183198
// {op_mapping} doesn't have a mapping for {old_index}. The assembler
184199
// should provide the mapping.
@@ -1294,8 +1309,7 @@ class GraphVisitor : public Next {
12941309
DCHECK(Asm().input_graph().BelongsToThisGraph(old_index));
12951310
DCHECK_IMPLIES(new_index.valid(),
12961311
Asm().output_graph().BelongsToThisGraph(new_index));
1297-
if constexpr (reducer_list_contains<typename Next::ReducerList,
1298-
VariableReducer>::value) {
1312+
if (contains_variable_reducer_) {
12991313
if (current_block_needs_variables_) {
13001314
MaybeVariable var = GetVariableFor(old_index);
13011315
if (!var.has_value()) {
@@ -1393,29 +1407,30 @@ template <template <class> class... Reducers>
13931407
class CopyingPhaseImpl {
13941408
public:
13951409
static void Run(Graph& input_graph, Zone* phase_zone,
1396-
bool trace_reductions = false) {
1410+
bool contains_variable_reducer, bool trace_reductions = false) {
13971411
TSAssembler<GraphVisitor, Reducers...> phase(
13981412
input_graph, input_graph.GetOrCreateCompanion(), phase_zone);
13991413
#ifdef DEBUG
14001414
if (trace_reductions) {
1401-
phase.template VisitGraph<true>();
1415+
phase.template VisitGraph<true>(contains_variable_reducer);
14021416
} else {
1403-
phase.template VisitGraph<false>();
1417+
phase.template VisitGraph<false>(contains_variable_reducer);
14041418
}
14051419
#else
1406-
phase.template VisitGraph<false>();
1420+
phase.template VisitGraph<false>(contains_variable_reducer);
14071421
#endif // DEBUG
14081422
}
14091423
};
14101424

14111425
template <template <typename> typename... Reducers>
14121426
class CopyingPhase {
14131427
public:
1428+
template <bool contains_variable_reducer>
14141429
static void Run(Zone* phase_zone) {
14151430
PipelineData& data = PipelineData::Get();
14161431
Graph& input_graph = data.graph();
14171432
CopyingPhaseImpl<Reducers...>::Run(
1418-
input_graph, phase_zone, data.info()->turboshaft_trace_reduction());
1433+
input_graph, phase_zone, contains_variable_reducer, data.info()->turboshaft_trace_reduction());
14191434
}
14201435
};
14211436

‎deps/v8/src/compiler/turboshaft/csa-optimize-phase.cc‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,23 +25,23 @@ namespace v8::internal::compiler::turboshaft {
2525
void CsaLoadEliminationPhase::Run(Zone* temp_zone) {
2626
CopyingPhase<VariableReducer, MachineOptimizationReducer,
2727
RequiredOptimizationReducer,
28-
ValueNumberingReducer>::Run(temp_zone);
28+
ValueNumberingReducer>::Run<true>(temp_zone);
2929

3030
CopyingPhase<VariableReducer, LateLoadEliminationReducer,
3131
MachineOptimizationReducer, RequiredOptimizationReducer,
32-
ValueNumberingReducer>::Run(temp_zone);
32+
ValueNumberingReducer>::Run<true>(temp_zone);
3333
}
3434

3535
void CsaLateEscapeAnalysisPhase::Run(Zone* temp_zone) {
3636
CopyingPhase<VariableReducer, LateEscapeAnalysisReducer,
3737
MachineOptimizationReducer, RequiredOptimizationReducer,
38-
ValueNumberingReducer>::Run(temp_zone);
38+
ValueNumberingReducer>::Run<true>(temp_zone);
3939
}
4040

4141
void CsaBranchEliminationPhase::Run(Zone* temp_zone) {
4242
CopyingPhase<VariableReducer, MachineOptimizationReducer,
4343
BranchEliminationReducer, RequiredOptimizationReducer,
44-
ValueNumberingReducer>::Run(temp_zone);
44+
ValueNumberingReducer>::Run<true>(temp_zone);
4545
}
4646

4747
void CsaOptimizePhase::Run(Zone* temp_zone) {
@@ -51,7 +51,7 @@ void CsaOptimizePhase::Run(Zone* temp_zone) {
5151
CopyingPhase<VariableReducer, PretenuringPropagationReducer,
5252
MachineOptimizationReducer, MemoryOptimizationReducer,
5353
RequiredOptimizationReducer,
54-
ValueNumberingReducer>::Run(temp_zone);
54+
ValueNumberingReducer>::Run<true>(temp_zone);
5555
}
5656

5757
} // namespace v8::internal::compiler::turboshaft

‎deps/v8/src/compiler/turboshaft/debug-feature-lowering-phase.cc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ namespace v8::internal::compiler::turboshaft {
1111

1212
void DebugFeatureLoweringPhase::Run(Zone* temp_zone) {
1313
#ifdef V8_ENABLE_DEBUG_CODE
14-
turboshaft::CopyingPhase<turboshaft::DebugFeatureLoweringReducer>::Run(
14+
turboshaft::CopyingPhase<turboshaft::DebugFeatureLoweringReducer>::Run<false>(
1515
temp_zone);
1616
#endif
1717
}

‎deps/v8/src/compiler/turboshaft/int64-lowering-phase.cc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ void Int64LoweringPhase::Run(Zone* temp_zone) {
1616
#if V8_TARGET_ARCH_32_BIT
1717
turboshaft::CopyingPhase<
1818
turboshaft::Int64LoweringReducer, turboshaft::VariableReducer,
19-
turboshaft::RequiredOptimizationReducer>::Run(temp_zone);
19+
turboshaft::RequiredOptimizationReducer>::Run<true>(temp_zone);
2020
#else
2121
UNREACHABLE();
2222
#endif

‎deps/v8/src/compiler/turboshaft/loop-peeling-phase.cc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ void LoopPeelingPhase::Run(Zone* temp_zone) {
2323
turboshaft::VariableReducer,
2424
turboshaft::MachineOptimizationReducer,
2525
turboshaft::RequiredOptimizationReducer,
26-
turboshaft::ValueNumberingReducer>::Run(temp_zone);
26+
turboshaft::ValueNumberingReducer>::Run<true>(temp_zone);
2727
}
2828

2929
} // namespace v8::internal::compiler::turboshaft

0 commit comments

Comments
 (0)