nodejs / node

Node.js JavaScript runtime ✨🐢🚀✨
https://nodejs.org
Other
107.87k stars 29.72k forks source link

[v12.x] --jitless not working on aix on v12.x #32600

Closed MylesBorins closed 4 years ago

MylesBorins commented 4 years ago

Refs: https://github.com/nodejs/node/pull/32594 Refs: https://github.com/nodejs/node/pull/32100

Tracking issue to make sure we get this fixed

richardlau commented 4 years ago

node --jitless coredumps on AIX in v12.x-staging but is okay on master. @nodejs/platform-aix is there something backportable to V8 7.8 to address this?

mhdawson commented 4 years ago

@miladfarca, @john-yan can you take a look.

miladfarca commented 4 years ago

Issue has to do with AIX function descriptors. This CL needs to be back-ported to V8 7.8: https://chromium-review.googlesource.com/c/v8/v8/+/1825961

miladfarca commented 4 years ago

PR for back-porting: https://github.com/nodejs/node/pull/32619

richardlau commented 4 years ago

Was addressed in https://github.com/nodejs/node/pull/32619.