v8: update v8 patch to avoid breaking building with clang

zcbenz · zcbenz · commit c3bc66a01ec3 · 2024-04-02T19:01:55.000+09:00
diff --git a/deps/v8/src/compiler/turboshaft/assembler.h b/deps/v8/src/compiler/turboshaft/assembler.h
@@ -3934,7 +3934,7 @@ class TSAssembler
     : public Assembler<reducer_list<TurboshaftAssemblerOpInterface, Reducers...,
                                     TSReducerBase>> {
  public:
-#ifdef _WIN32
+#if defined(_MSC_VER) && !defined(__clang__)
   explicit TSAssembler(Graph& input_graph, Graph& output_graph,
                        Zone* phase_zone)
       : Assembler(input_graph, output_graph, phase_zone) {}
diff --git a/deps/v8/src/compiler/turboshaft/copying-phase.h b/deps/v8/src/compiler/turboshaft/copying-phase.h
@@ -193,7 +193,12 @@ class GraphVisitor : public Next {
     DCHECK(old_index.valid());
     OpIndex result = op_mapping_[old_index];
 
+#if defined(_MSC_VER) && !defined(__clang__)
     if (contains_variable_reducer_) {
+#else
+    if constexpr (reducer_list_contains<typename Next::ReducerList,
+                                        VariableReducer>::value) {
+#endif
       if (!result.valid()) {
         // {op_mapping} doesn't have a mapping for {old_index}. The assembler
         // should provide the mapping.
@@ -1309,7 +1314,12 @@ class GraphVisitor : public Next {
     DCHECK(Asm().input_graph().BelongsToThisGraph(old_index));
     DCHECK_IMPLIES(new_index.valid(),
                    Asm().output_graph().BelongsToThisGraph(new_index));
+#if defined(_MSC_VER) && !defined(__clang__)
     if (contains_variable_reducer_) {
+#else
+    if constexpr (reducer_list_contains<typename Next::ReducerList,
+                                        VariableReducer>::value) {
+#endif
       if (current_block_needs_variables_) {
         MaybeVariable var = GetVariableFor(old_index);
         if (!var.has_value()) {
