deps: fix V8 build issue with inline methods

gengjiawen · targos · commit 0d78bc310140 · 2021-03-15T15:55:52.000+01:00
PR-URL: #35415 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com>
diff --git a/common.gypi b/common.gypi
@@ -36,7 +36,7 @@
 
     # Reset this number to 0 on major V8 upgrades.
     # Increment by one for each non-official patch applied to deps/v8.
-    'v8_embedder_string': '-node.4',
+    'v8_embedder_string': '-node.5',
 
     ##### V8 defaults for Node.js #####
 
diff --git a/deps/v8/src/objects/fixed-array-inl.h b/deps/v8/src/objects/fixed-array-inl.h
@@ -88,13 +88,15 @@ bool FixedArray::is_the_hole(Isolate* isolate, int index) {
   return get(isolate, index).IsTheHole(isolate);
 }
 
+#if !defined(_WIN32) || defined(_WIN64)
 void FixedArray::set(int index, Smi value) {
   DCHECK_NE(map(), GetReadOnlyRoots().fixed_cow_array_map());
   DCHECK_LT(static_cast<unsigned>(index), static_cast<unsigned>(length()));
   DCHECK(Object(value).IsSmi());
   int offset = OffsetOfElementAt(index);
   RELAXED_WRITE_FIELD(*this, offset, value);
 }
+#endif
 
 void FixedArray::set(int index, Object value) {
   DCHECK_NE(GetReadOnlyRoots().fixed_cow_array_map(), map());
diff --git a/deps/v8/src/objects/fixed-array.h b/deps/v8/src/objects/fixed-array.h
@@ -130,7 +130,18 @@ class FixedArray
   inline bool is_the_hole(Isolate* isolate, int index);
 
   // Setter that doesn't need write barrier.
+#if defined(_WIN32) && !defined(_WIN64)
+  inline void set(int index, Smi value) {
+    DCHECK_NE(map(), GetReadOnlyRoots().fixed_cow_array_map());
+    DCHECK_LT(static_cast<unsigned>(index), static_cast<unsigned>(length()));
+    DCHECK(Object(value).IsSmi());
+    int offset = OffsetOfElementAt(index);
+    RELAXED_WRITE_FIELD(*this, offset, value);
+  }
+#else
   inline void set(int index, Smi value);
+#endif
+
   // Setter with explicit barrier mode.
   inline void set(int index, Object value, WriteBarrierMode mode);
 
diff --git a/deps/v8/src/objects/tagged-field.h b/deps/v8/src/objects/tagged-field.h
@@ -48,7 +48,7 @@ class TaggedField : public AllStatic {
                                int offset = 0);
 
   static inline void Relaxed_Store(HeapObject host, T value);
-  static inline void Relaxed_Store(HeapObject host, int offset, T value);
+  static void Relaxed_Store(HeapObject host, int offset, T value);
 
   static inline T Acquire_Load(HeapObject host, int offset = 0);
   static inline T Acquire_Load(IsolateRoot isolate, HeapObject host,

Original file line number	Diff line number	Diff line change
`@@ -88,13 +88,15 @@ bool FixedArray::is_the_hole(Isolate* isolate, int index) {`
`88`	`88`	`return get(isolate, index).IsTheHole(isolate);`
`89`	`89`	`}`
`90`	`90`
	`91`	`+#if !defined(_WIN32) \|\| defined(_WIN64)`
`91`	`92`	`void FixedArray::set(int index, Smi value) {`
`92`	`93`	`DCHECK_NE(map(), GetReadOnlyRoots().fixed_cow_array_map());`
`93`	`94`	`DCHECK_LT(static_cast<unsigned>(index), static_cast<unsigned>(length()));`
`94`	`95`	`DCHECK(Object(value).IsSmi());`
`95`	`96`	`int offset = OffsetOfElementAt(index);`
`96`	`97`	`RELAXED_WRITE_FIELD(*this, offset, value);`
`97`	`98`	`}`
	`99`	`+#endif`
`98`	`100`
`99`	`101`	`void FixedArray::set(int index, Object value) {`
`100`	`102`	`DCHECK_NE(GetReadOnlyRoots().fixed_cow_array_map(), map());`