diff options
author | Matheus Marchini <mmarchini@netflix.com> | 2020-03-10 16:58:15 -0700 |
---|---|---|
committer | Michaël Zasso <targos@protonmail.com> | 2020-11-15 16:47:19 +0100 |
commit | f4b4e21b2f7d72e1a485fab03246b5d7a00e86eb (patch) | |
tree | f12ae1bca54787d359e4a0daba4bca6e7f3fecff /deps | |
parent | f6a84540d8285c7419016adbc1cfb2b56e394870 (diff) | |
download | ios-node-v8-f4b4e21b2f7d72e1a485fab03246b5d7a00e86eb.tar.gz ios-node-v8-f4b4e21b2f7d72e1a485fab03246b5d7a00e86eb.tar.bz2 ios-node-v8-f4b4e21b2f7d72e1a485fab03246b5d7a00e86eb.zip |
deps: patch V8 to run on Xcode 8
Patch V8 (wasm/wasm-module.cc) to remove const qualifier from type
passed to template call of `OwnedVector::Of`. Xcode 8 can't convert
'OwnedVector<unsigned char>' to 'OwnedVector<const unsigned char>' when
returning from a function (which is likely a bug on Xcode, considering
this worked on the prior version of Xcode as well as newer versions).
This workaround shouldn't affect the application, since the const
qualifier is preserved in the AsmJsOffsetInformation::encoded_offset_.
There's also a V8 test passing a const-qualified type to ::Of, but since
we don't test V8 on Xcode 8, it should be fine to leave it as is.
Signed-off-by: Matheus Marchini <mmarchini@netflix.com>
PR-URL: https://github.com/nodejs/node/pull/32116
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'deps')
-rw-r--r-- | deps/v8/src/wasm/wasm-module.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/deps/v8/src/wasm/wasm-module.cc b/deps/v8/src/wasm/wasm-module.cc index 51b2ebcd93..3d935f27be 100644 --- a/deps/v8/src/wasm/wasm-module.cc +++ b/deps/v8/src/wasm/wasm-module.cc @@ -137,7 +137,7 @@ void LazilyGeneratedNames::AddForTesting(int function_index, AsmJsOffsetInformation::AsmJsOffsetInformation( Vector<const byte> encoded_offsets) - : encoded_offsets_(OwnedVector<const uint8_t>::Of(encoded_offsets)) {} + : encoded_offsets_(OwnedVector<uint8_t>::Of(encoded_offsets)) {} AsmJsOffsetInformation::~AsmJsOffsetInformation() = default; |