Commit a19e9bd
tools: patch V8 to run on older XCode versions
Patch V8 (compiler/js-heap-broker.cc) to remove the use of an optional
property, which is a fairly new C++ feature, since that requires a newer
XCode version than the minimum requirement in BUILDING.md and thus
breaks CI.
Backport-PR-URL: #30109
PR-URL: #29694
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>1 parent b4e8f0d commit a19e9bd
2 files changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -4115,6 +4115,7 @@ GlobalAccessFeedback::GlobalAccessFeedback(PropertyCellRef cell, | |||
| 4115 | 4115 | ||
| 4116 | 4116 | GlobalAccessFeedback::GlobalAccessFeedback(FeedbackSlotKind slot_kind) | |
| 4117 | 4117 | : ProcessedFeedback(kGlobalAccess, slot_kind), | |
| 4118 | + cell_or_context_(base::nullopt), | ||
| 4118 | 4119 | index_and_immutable_(0 /* doesn't matter */) { | |
| 4119 | 4120 | DCHECK(IsGlobalICKind(slot_kind)); | |
| 4120 | 4121 | } | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -280,7 +280,7 @@ Interpreter::NewSourcePositionCollectionJob( | |||
| 280 | 280 | auto job = base::make_unique<InterpreterCompilationJob>(parse_info, literal, | |
| 281 | 281 | allocator, nullptr); | |
| 282 | 282 | job->compilation_info()->SetBytecodeArray(existing_bytecode); | |
| 283 | - return job; | ||
| 283 | + return std::unique_ptr<UnoptimizedCompilationJob> { static_cast<UnoptimizedCompilationJob*>(job.release()) }; | ||
| 284 | 284 | } | |
| 285 | 285 | ||
| 286 | 286 | void Interpreter::ForEachBytecode( | |
0 commit comments