FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Revert "deps: V8: cherry-pick 7107287" · nodejs/node@b7fab70 · GitHub

/ node Public

Commit b7fab70

Browse files
richardlau
authored and
RafaelGSS
committed
Revert "deps: V8: cherry-pick 7107287"
This reverts commit 142b593. PR-URL: #62894 Reviewed-By: René <contact.9a5d6388@renegade334.me.uk> Reviewed-By: Abdirahim Musse <abdirahim.musse@ibm.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> (cherry picked from commit c028766)
1 parent ecf8721 commit b7fab70

6 files changed

Lines changed: 12 additions & 33 deletions

File tree

‎common.gypi‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838

3939
# Reset this number to 0 on major V8 upgrades.
4040
# Increment by one for each non-official patch applied to deps/v8.
41-
'v8_embedder_string': '-node.18',
41+
'v8_embedder_string': '-node.17',
4242

4343
##### V8 defaults for Node.js #####
4444

‎deps/v8/BUILD.gn‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1627,7 +1627,7 @@ config("toolchain") {
16271627
if (v8_current_cpu == "ppc64") {
16281628
defines += [ "V8_TARGET_ARCH_PPC64" ]
16291629
cflags += [ "-ffp-contract=off" ]
1630-
if (current_os == "aix" and !is_clang) {
1630+
if (current_os == "aix") {
16311631
cflags += [
16321632
# Work around AIX ceil, trunc and round oddities.
16331633
"-mcpu=power5+",

‎deps/v8/src/builtins/ppc/builtins-ppc.cc‎

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -4201,19 +4201,6 @@ void Builtins::Generate_CEntry(MacroAssembler* masm, int result_size,
42014201

42024202
// If return value is on the stack, pop it to registers.
42034203
if (needs_return_buffer) {
4204-
Label done;
4205-
if (switch_to_central_stack) {
4206-
Label no_stack_change;
4207-
__ CmpU64(kOldSPRegister, Operand(0), r0);
4208-
__ beq(&no_stack_change);
4209-
__ addi(r3, kOldSPRegister,
4210-
Operand((kStackFrameExtraParamSlot + 1) * kSystemPointerSize));
4211-
__ b(&done);
4212-
__ bind(&no_stack_change);
4213-
}
4214-
__ addi(r3, sp,
4215-
Operand((kStackFrameExtraParamSlot + 1) * kSystemPointerSize));
4216-
__ bind(&done);
42174204
__ LoadU64(r4, MemOperand(r3, kSystemPointerSize));
42184205
__ LoadU64(r3, MemOperand(r3));
42194206
}

‎deps/v8/src/compiler/turboshaft/operations.h‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -525,7 +525,7 @@ class InputsRepFactory {
525525
};
526526
};
527527

528-
struct __attribute__((packed)) EffectDimensions {
528+
struct EffectDimensions {
529529
// Produced by loads, consumed by operations that should not move before loads
530530
// because they change memory.
531531
bool load_heap_memory : 1;
@@ -620,7 +620,7 @@ static_assert(sizeof(EffectDimensions) == sizeof(EffectDimensions::Bits));
620620
// they become more restricted in their movement. Note that calls are not the
621621
// most side-effectful operations, as they do not leave the heap in an
622622
// inconsistent state, so they do not need to be marked as raw heap access.
623-
struct __attribute__((packed)) OpEffects {
623+
struct OpEffects {
624624
EffectDimensions produces;
625625
EffectDimensions consumes;
626626

@@ -2904,7 +2904,7 @@ struct ConstantOp : FixedArityOperationT<0, ConstantOp> {
29042904
// When result_rep is RegisterRepresentation::Compressed(), then the load does
29052905
// not decompress the value.
29062906
struct LoadOp : OperationT<LoadOp> {
2907-
struct __attribute__((packed)) Kind {
2907+
struct Kind {
29082908
// The `base` input is a tagged pointer to a HeapObject.
29092909
bool tagged_base : 1;
29102910
// The effective address might be unaligned. This is only set to true if

‎deps/v8/src/execution/simulator.h‎

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -199,17 +199,13 @@ class GeneratedCode {
199199
return fn(args...);
200200
#else
201201
// AIX ABI requires function descriptors (FD). Artificially create a pseudo
202-
// FD to ensure correct dispatch to generated code.
203-
void* function_desc[3];
204-
Signature* fn;
205-
asm("std %1, 0(%2)\n\t"
206-
"li 0, 0\n\t"
207-
"std 0, 8(%2)\n\t"
208-
"std 0, 16(%2)\n\t"
209-
"mr %0, %2\n\t"
210-
: "=r"(fn)
211-
: "r"(fn_ptr_), "r"(function_desc)
212-
: "memory", "0");
202+
// FD to ensure correct dispatch to generated code. The 'volatile'
203+
// declaration is required to avoid the compiler from not observing the
204+
// alias of the pseudo FD to the function pointer, and hence, optimizing the
205+
// pseudo FD declaration/initialization away.
206+
volatile Address function_desc[] = {reinterpret_cast<Address>(fn_ptr_), 0,
207+
0};
208+
Signature* fn = reinterpret_cast<Signature*>(function_desc);
213209
return fn(args...);
214210
#endif // V8_OS_ZOS
215211
#else

‎deps/v8/src/trap-handler/handler-shared.cc‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,7 @@ namespace v8 {
2424
namespace internal {
2525
namespace trap_handler {
2626

27-
#if defined(V8_OS_AIX)
28-
__thread bool TrapHandlerGuard::is_active_ = 0;
29-
#else
3027
thread_local bool TrapHandlerGuard::is_active_ = 0;
31-
#endif
3228

3329
size_t gNumCodeObjects = 0;
3430
CodeProtectionInfoListEntry* gCodeObjects = nullptr;

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL