Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

failures on aix v6.x-staging #218

Closed
MylesBorins opened this issue May 16, 2017 · 9 comments
Closed

failures on aix v6.x-staging #218

MylesBorins opened this issue May 16, 2017 · 9 comments

Comments

@MylesBorins
Copy link
Contributor

After landing libuv update we are seeing failures on aix during compilation

/cc @nodejs/platform-aix

https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/5891/console

@MylesBorins
Copy link
Contributor Author

we may need to land nodejs/node#10085, but this is unrelated to the compiler failure

/cc @mhdawson

@MylesBorins
Copy link
Contributor Author

It looks like the failure is specific to two member function not existing in v8::internal

../deps/v8/src/builtins/ppc/builtins-ppc.cc:672:76: error: no 'void v8::internal::Builtins::Generate_JSBuiltinsConstructStubForBase(v8::internal::MacroAssembler*)' member function declared in class 'v8::internal::Builtins'
 void Builtins::Generate_JSBuiltinsConstructStubForBase(MacroAssembler* masm) {
                                                                            ^
../deps/v8/src/builtins/ppc/builtins-ppc.cc:677:25: error: no 'void v8::internal::Builtins::Generate_JSBuiltinsConstructStubForDerived(v8::internal::MacroAssembler*)' member function declared in class 'v8::internal::Builtins'
     MacroAssembler* masm) {
                         ^
../deps/v8/src/builtins/ppc/builtins-ppc.cc: In static member function 'static void v8::internal::Builtins::Generate_CompileLazy(v8::internal::MacroAssembler*)':
../deps/v8/src/builtins/ppc/builtins-ppc.cc:1359:33: error: 'kOptimizedCodeMapOffset' is not a member of 'v8::internal::SharedFunctionInfo'
            FieldMemOperand(map, SharedFunctionInfo::kOptimizedCodeMapOffset));

@MylesBorins
Copy link
Contributor Author

/cc @nodejs/v8

@jeisinger
Copy link

that might be because the ppc port didn't yet port over https://chromium-review.googlesource.com/c/489525/

@rmcilroy
Copy link

Actually from looking at the errors I think it's because they've not yet ported over https://chromium-review.googlesource.com/c/500134/. They did port it over but reverted it when the original CL was reverted - shouldn't be too difficult to reland with the updates.

@jbajwa
Copy link

jbajwa commented May 16, 2017

I'm working on porting to V8 master. I was wondering how it is affecting v6.x-staging branch where V8 is at 5.1 and the original CL was committed to V8 6.0 ?

@jbajwa
Copy link

jbajwa commented May 16, 2017

The test job linked above seems to be from the canary branch,

Started by upstream project "node-test-commit-aix" build number 5891
originally caused by:
 Started by upstream project "node-test-commit-node-v8" build number 16
 originally caused by:
  Started by upstream project "node-update-v8-canary" build number 30
  originally caused by:
   Started by timer

@targos
Copy link
Member

targos commented May 16, 2017

This failure is from the node-update-v8-canary job. It shouldn't be related to v6.x-staging.

@MylesBorins
Copy link
Contributor Author

OH LOL... I messed that up This is unrelated to staging. Sorry for the fire drill 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants