Skip to content

Commit 6e150f9

Browse files
targosnodejs-github-bot
authored andcommitted
deps: V8: cherry-pick 97199f686e2f
Original commit message: aix: add work around for f16 type AIX builds started to fail after this commit https://chromium.googlesource.com/v8/v8.git/+/d057564707d3a5df074b7f49a12a2f1e96638f94. Change-Id: I25a5c4ae3b4fe5c27a9fb9e35e2bcd2bbed40351 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5789180 Reviewed-by: Clemens Backes <clemensb@chromium.org> Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#95952} Refs: v8/v8@97199f6 PR-URL: #54536 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
1 parent 1e16779 commit 6e150f9

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

common.gypi

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636

3737
# Reset this number to 0 on major V8 upgrades.
3838
# Increment by one for each non-official patch applied to deps/v8.
39-
'v8_embedder_string': '-node.8',
39+
'v8_embedder_string': '-node.9',
4040

4141
##### V8 defaults for Node.js #####
4242

deps/v8/src/utils/utils.h

+9
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@
2727

2828
#if defined(V8_OS_AIX)
2929
#include <fenv.h> // NOLINT(build/c++11)
30+
31+
#include "src/wasm/float16.h"
3032
#endif
3133

3234
#ifdef _MSC_VER
@@ -814,6 +816,13 @@ T FpOpWorkaround(T input, T value) {
814816
}
815817
return value;
816818
}
819+
820+
template <>
821+
inline Float16 FpOpWorkaround(Float16 input, Float16 value) {
822+
float result = FpOpWorkaround(input.ToFloat32(), value.ToFloat32());
823+
return Float16::FromFloat32(result);
824+
}
825+
817826
#endif
818827

819828
V8_EXPORT_PRIVATE bool PassesFilter(base::Vector<const char> name,

0 commit comments

Comments
 (0)