Skip to content

Commit 1a7a0e3

Browse files
joyeecheungnodejs-github-bot
joyeecheung
authored and
nodejs-github-bot
committed
deps: fix building v8 on macos 13 and Apple Clang 14
1 parent bcca195 commit 1a7a0e3

File tree

10 files changed

+79
-69
lines changed

10 files changed

+79
-69
lines changed

‎deps/v8/src/compiler/turboshaft/string-escape-analysis-reducer.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ class StringEscapeAnalysisReducer : public Next {
149149
}
150150

151151
private:
152-
ElidedStringPart(Kind kind, V<String> index) : data(index), kind(kind) {}
152+
ElidedStringPart(Kind kind, V<String> index) : data{index}, kind(kind) {}
153153
};
154154

155155
void Analyze() {

‎deps/v8/src/heap/heap-visitor-inl.h‎

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -93,26 +93,23 @@ Tagged<T> HeapVisitor<ConcreteVisitor>::Cast(Tagged<HeapObject> object,
9393
}
9494

9595
template <typename ConcreteVisitor>
96-
size_t HeapVisitor<ConcreteVisitor>::Visit(Tagged<HeapObject> object)
97-
requires(!ConcreteVisitor::UsePrecomputedObjectSize())
98-
{
96+
template <typename T, typename>
97+
size_t HeapVisitor<ConcreteVisitor>::Visit(Tagged<HeapObject> object) {
9998
return Visit(object->map(cage_base()), object);
10099
}
101100

102101
template <typename ConcreteVisitor>
102+
template <typename T, typename>
103103
size_t HeapVisitor<ConcreteVisitor>::Visit(Tagged<Map> map,
104-
Tagged<HeapObject> object)
105-
requires(!ConcreteVisitor::UsePrecomputedObjectSize())
106-
{
104+
Tagged<HeapObject> object) {
107105
return Visit(map, object, MaybeObjectSize());
108106
}
109107

110108
template <typename ConcreteVisitor>
109+
template <typename T, typename>
111110
size_t HeapVisitor<ConcreteVisitor>::Visit(Tagged<Map> map,
112111
Tagged<HeapObject> object,
113-
int object_size)
114-
requires(ConcreteVisitor::UsePrecomputedObjectSize())
115-
{
112+
int object_size) {
116113
return Visit(map, object, MaybeObjectSize(object_size));
117114
}
118115

‎deps/v8/src/heap/heap-visitor.h‎

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -196,15 +196,18 @@ class HeapVisitor : public ObjectVisitorWithCageBases {
196196
inline explicit HeapVisitor(Isolate* isolate);
197197
inline explicit HeapVisitor(Heap* heap);
198198

199-
V8_INLINE size_t Visit(Tagged<HeapObject> object)
200-
requires(!ConcreteVisitor::UsePrecomputedObjectSize());
199+
template <typename T = ConcreteVisitor,
200+
typename = std::enable_if_t<!T::UsePrecomputedObjectSize()>>
201+
V8_INLINE size_t Visit(Tagged<HeapObject> object);
201202

202-
V8_INLINE size_t Visit(Tagged<Map> map, Tagged<HeapObject> object)
203-
requires(!ConcreteVisitor::UsePrecomputedObjectSize());
203+
template <typename T = ConcreteVisitor,
204+
typename = std::enable_if_t<!T::UsePrecomputedObjectSize()>>
205+
V8_INLINE size_t Visit(Tagged<Map> map, Tagged<HeapObject> object);
204206

207+
template <typename T = ConcreteVisitor,
208+
typename = std::enable_if_t<T::UsePrecomputedObjectSize()>>
205209
V8_INLINE size_t Visit(Tagged<Map> map, Tagged<HeapObject> object,
206-
int object_size)
207-
requires(ConcreteVisitor::UsePrecomputedObjectSize());
210+
int object_size);
208211

209212
protected:
210213
V8_INLINE size_t Visit(Tagged<Map> map, Tagged<HeapObject> object,

‎deps/v8/src/heap/mark-compact.cc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3863,7 +3863,7 @@ void MarkCompactCollector::ClearWeakCollections() {
38633863

38643864
template <typename TObjectAndSlot, typename TMaybeSlot>
38653865
void MarkCompactCollector::ClearWeakReferences(
3866-
WeakObjects::WeakObjectWorklist<TObjectAndSlot>::Local& worklist,
3866+
typename WeakObjects::WeakObjectWorklist<TObjectAndSlot>::Local& worklist,
38673867
Tagged<HeapObjectReference> cleared_weak_ref) {
38683868
TObjectAndSlot slot;
38693869
while (worklist.Pop(&slot)) {

‎deps/v8/src/heap/mark-compact.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ class MarkCompactCollector final {
339339
// Common implementation of the above two.
340340
template <typename TObjectAndSlot, typename TMaybeSlot>
341341
void ClearWeakReferences(
342-
WeakObjects::WeakObjectWorklist<TObjectAndSlot>::Local& worklist,
342+
typename WeakObjects::WeakObjectWorklist<TObjectAndSlot>::Local& worklist,
343343
Tagged<HeapObjectReference> cleared_weak_ref);
344344

345345
// Goes through the list of encountered non-trivial weak references and

‎deps/v8/src/json/json-stringifier.cc‎

Lines changed: 32 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -2078,7 +2078,7 @@ template <typename Char>
20782078
template <typename StringT>
20792079
FastJsonStringifierResult FastJsonStringifier<Char>::SerializeObjectKey(
20802080
Tagged<String> obj, bool comma, const DisallowGarbageCollection& no_gc) {
2081-
using StringChar = StringT::Char;
2081+
using StringChar = typename StringT::Char;
20822082
if constexpr (is_one_byte && sizeof(StringChar) == 2) {
20832083
return CHANGE_ENCODING;
20842084
} else {
@@ -2104,7 +2104,7 @@ template <typename StringT, bool deferred_key>
21042104
FastJsonStringifierResult FastJsonStringifier<Char>::SerializeString(
21052105
Tagged<HeapObject> obj, bool comma, Tagged<String> key,
21062106
const DisallowGarbageCollection& no_gc) {
2107-
using StringChar = StringT::Char;
2107+
using StringChar = typename StringT::Char;
21082108
if constexpr (is_one_byte && sizeof(StringChar) == 2) {
21092109
return CHANGE_ENCODING;
21102110
} else {
@@ -2283,36 +2283,39 @@ FastJsonStringifierResult FastJsonStringifier<Char>::SerializeJSObject(
22832283
break;
22842284
case UNDEFINED:
22852285
break;
2286-
case UNCHANGED:
2287-
stack_.emplace_back(ContinuationRecord::kObject, obj,
2288-
i.as_uint32() + 1);
2286+
case UNCHANGED: {
2287+
ContinuationRecord rec1{ContinuationRecord::kObject, obj, i.as_uint32() + 1};
2288+
stack_.emplace_back(std::move(rec1));
22892289
// property can be an object or array. We don't need to distinguish
22902290
// as index is 0 anyways.
2291-
stack_.emplace_back(ContinuationRecord::kObject, property, 0);
2291+
ContinuationRecord rec2{ContinuationRecord::kObject, property, 0};
2292+
stack_.emplace_back(std::move(rec2));
22922293
result = SerializeObjectKey(key_name, comma, no_gc);
22932294
if constexpr (is_one_byte) {
22942295
if (V8_UNLIKELY(result != SUCCESS)) {
22952296
DCHECK_EQ(result, CHANGE_ENCODING);
2296-
stack_.emplace_back(ContinuationRecord::kObjectKey, key_name,
2297-
comma);
2297+
ContinuationRecord rec3{ContinuationRecord::kObjectKey, key_name, comma};
2298+
stack_.emplace_back(std::move(rec3));
22982299
return result;
22992300
}
23002301
}
23012302
return result;
2302-
case CHANGE_ENCODING:
2303+
}
2304+
case CHANGE_ENCODING: {
23032305
DCHECK(is_one_byte);
2304-
stack_.emplace_back(ContinuationRecord::kObject, obj,
2305-
i.as_uint32() + 1);
2306-
stack_.emplace_back(ContinuationRecord::kResumeFromOther, property,
2307-
0);
2306+
ContinuationRecord rec1{ContinuationRecord::kObject, obj, i.as_uint32() + 1};
2307+
stack_.emplace_back(std::move(rec1));
2308+
ContinuationRecord rec2{ContinuationRecord::kResumeFromOther, property, 0};
2309+
stack_.emplace_back(std::move(rec2));
23082310
result = SerializeObjectKey(key_name, comma, no_gc);
23092311
if (V8_UNLIKELY(result != SUCCESS)) {
2310-
stack_.emplace_back(ContinuationRecord::kObjectKey, key_name,
2311-
comma);
2312+
ContinuationRecord rec3{ContinuationRecord::kObjectKey, key_name, comma};
2313+
stack_.emplace_back(std::move(rec3));
23122314
return result;
23132315
}
23142316
DCHECK(IsString(property));
23152317
return result;
2318+
}
23162319
case SLOW_PATH:
23172320
case EXCEPTION:
23182321
return result;
@@ -2455,15 +2458,21 @@ FastJsonStringifierResult FastJsonStringifier<Char>::SerializeFixedArrayElement(
24552458
case UNDEFINED:
24562459
AppendCStringLiteral("null");
24572460
return SUCCESS;
2458-
case CHANGE_ENCODING:
2461+
case CHANGE_ENCODING: {
24592462
DCHECK(IsString(obj));
2460-
stack_.emplace_back(ContinuationRecord::kArray, array, i + 1);
2461-
stack_.emplace_back(ContinuationRecord::kResumeFromOther, obj, 0);
2463+
ContinuationRecord rec1{ContinuationRecord::kArray, array, i + 1};
2464+
stack_.emplace_back(std::move(rec1));
2465+
ContinuationRecord rec2{ContinuationRecord::kResumeFromOther, obj, 0};
2466+
stack_.emplace_back(std::move(rec2));
24622467
return result;
2463-
case UNCHANGED:
2464-
stack_.emplace_back(ContinuationRecord::kArray, array, i + 1);
2465-
stack_.emplace_back(ContinuationRecord::kResumeFromOther, obj, 0);
2468+
}
2469+
case UNCHANGED: {
2470+
ContinuationRecord rec1{ContinuationRecord::kArray, array, i + 1};
2471+
stack_.emplace_back(std::move(rec1));
2472+
ContinuationRecord rec2{ContinuationRecord::kResumeFromOther, obj, 0};
2473+
stack_.emplace_back(std::move(rec2));
24662474
return result;
2475+
}
24672476
default:
24682477
return result;
24692478
}
@@ -2540,7 +2549,8 @@ FastJsonStringifierResult FastJsonStringifier<Char>::SerializeObject(
25402549
if constexpr (is_one_byte) {
25412550
if (result == CHANGE_ENCODING) {
25422551
DCHECK(IsString(object));
2543-
stack_.emplace_back(ContinuationRecord::kResumeFromOther, object, 0);
2552+
ContinuationRecord rec{ContinuationRecord::kResumeFromOther, object, 0};
2553+
stack_.emplace_back(std::move(rec));
25442554
return result;
25452555
}
25462556
} else {

‎deps/v8/src/maglev/maglev-ir.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4197,7 +4197,7 @@ class CheckedNumberOrOddballToFloat64OrHoleyFloat64
41974197

41984198
private:
41994199
using TaggedToFloat64ConversionTypeOffset =
4200-
Base::template NextBitField<TaggedToFloat64ConversionType, 2>;
4200+
typename Base::template NextBitField<TaggedToFloat64ConversionType, 2>;
42014201
};
42024202

42034203
class CheckedNumberOrOddballToFloat64

‎deps/v8/src/objects/objects.h‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -604,11 +604,11 @@ class Object : public AllStatic {
604604
ConvertToInteger(Isolate* isolate, HandleType<Object> input);
605605
template <template <typename> typename HandleType>
606606
requires(std::is_convertible_v<HandleType<Object>, DirectHandle<Object>>)
607-
V8_WARN_UNUSED_RESULT static HandleType<Number>::MaybeType ConvertToInt32(
607+
V8_WARN_UNUSED_RESULT static typename HandleType<Number>::MaybeType ConvertToInt32(
608608
Isolate* isolate, HandleType<Object> input);
609609
template <template <typename> typename HandleType>
610610
requires(std::is_convertible_v<HandleType<Object>, DirectHandle<Object>>)
611-
V8_WARN_UNUSED_RESULT static HandleType<Number>::MaybeType ConvertToUint32(
611+
V8_WARN_UNUSED_RESULT static typename HandleType<Number>::MaybeType ConvertToUint32(
612612
Isolate* isolate, HandleType<Object> input);
613613
V8_EXPORT_PRIVATE V8_WARN_UNUSED_RESULT static MaybeHandle<Number>
614614
ConvertToLength(Isolate* isolate, DirectHandle<Object> input);

‎deps/v8/src/objects/smi.h‎

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,29 @@ namespace internal {
2424
// Smi stands for small integer.
2525
class Smi : public AllStatic {
2626
public:
27+
// Returns whether value can be represented in a Smi.
28+
template <typename T>
29+
static inline bool constexpr IsValid(T value)
30+
requires(std::is_integral_v<T> && std::is_signed_v<T>)
31+
{
32+
DCHECK_EQ(Internals::IsValidSmi(value),
33+
value >= kMinValue && value <= kMaxValue);
34+
return Internals::IsValidSmi(value);
35+
}
36+
template <typename T>
37+
static inline bool constexpr IsValid(T value)
38+
requires(std::is_integral_v<T> && std::is_unsigned_v<T>)
39+
{
40+
DCHECK_EQ(Internals::IsValidSmi(value), value <= kMaxValue);
41+
return Internals::IsValidSmi(value);
42+
}
43+
44+
// Convert a value to a Smi object.
45+
static inline constexpr Tagged<Smi> FromInt(int value) {
46+
DCHECK(Smi::IsValid(value));
47+
return Tagged<Smi>(Internals::IntegralToSmi(value));
48+
}
49+
2750
static inline constexpr Tagged<Smi> ToUint32Smi(Tagged<Smi> smi) {
2851
if (smi.value() <= 0) return Smi::FromInt(0);
2952
return Smi::FromInt(static_cast<uint32_t>(smi.value()));
@@ -34,12 +57,6 @@ class Smi : public AllStatic {
3457
return Tagged<Smi>(object.ptr()).value();
3558
}
3659

37-
// Convert a value to a Smi object.
38-
static inline constexpr Tagged<Smi> FromInt(int value) {
39-
DCHECK(Smi::IsValid(value));
40-
return Tagged<Smi>(Internals::IntegralToSmi(value));
41-
}
42-
4360
static inline constexpr Tagged<Smi> FromIntptr(intptr_t value) {
4461
DCHECK(Smi::IsValid(value));
4562
int smi_shift_bits = kSmiTagSize + kSmiShiftSize;
@@ -62,23 +79,6 @@ class Smi : public AllStatic {
6279
return FromInt(static_cast<int>(value));
6380
}
6481

65-
// Returns whether value can be represented in a Smi.
66-
template <typename T>
67-
static inline bool constexpr IsValid(T value)
68-
requires(std::is_integral_v<T> && std::is_signed_v<T>)
69-
{
70-
DCHECK_EQ(Internals::IsValidSmi(value),
71-
value >= kMinValue && value <= kMaxValue);
72-
return Internals::IsValidSmi(value);
73-
}
74-
template <typename T>
75-
static inline bool constexpr IsValid(T value)
76-
requires(std::is_integral_v<T> && std::is_unsigned_v<T>)
77-
{
78-
DCHECK_EQ(Internals::IsValidSmi(value), value <= kMaxValue);
79-
return Internals::IsValidSmi(value);
80-
}
81-
8282
// Compare two Smis x, y as if they were converted to strings and then
8383
// compared lexicographically. Returns:
8484
// -1 if x < y.

‎deps/v8/src/sandbox/external-entity-table.h‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ class V8_EXPORT_PRIVATE ExternalEntityTable
4848
: public SegmentedTable<Entry, size> {
4949
protected:
5050
using Base = SegmentedTable<Entry, size>;
51-
using FreelistHead = Base::FreelistHead;
52-
using Segment = Base::Segment;
53-
using WriteIterator = Base::WriteIterator;
51+
using FreelistHead = typename Base::FreelistHead;
52+
using Segment = typename Base::Segment;
53+
using WriteIterator = typename Base::WriteIterator;
5454
static constexpr size_t kSegmentSize = Base::kSegmentSize;
5555
static constexpr size_t kEntriesPerSegment = Base::kEntriesPerSegment;
5656
static constexpr size_t kEntrySize = Base::kEntrySize;

0 commit comments

Comments
 (0)