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

Back-port v8 changes for ppc64, Aix platform by V-for-Vasili · Pull Request #23695 · nodejs/node · GitHub

/ node Public

Back-port v8 changes for ppc64, Aix platform - #23695

Closed
V-for-Vasili wants to merge 6 commits into
nodejs:v10.x-stagingfrom
V-for-Vasili:Node-10-aix-backport-cherry-pick
ConversationCommits (6)ChecksFiles changed
Closed

Back-port v8 changes for ppc64, Aix platform#23695
V-for-Vasili wants to merge 6 commits into
nodejs:v10.x-stagingfrom
V-for-Vasili:Node-10-aix-backport-cherry-pick

Conversation

V-for-Vasili commented Oct 16, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor

Back-port v8 changes for ppc64, Aix platform
Changes included:

s390, ppc64: Enable v8gen.py on Linux s390, ppc64
V8 Revision: abab9fbb643e97d1fa7ddee9bfc609e0e4a4f184
Link: https://chromium-review.googlesource.com/c/v8/v8/+/1135540

fix gn builds on aix.
V8 Revision: d9e78322d0370543342d52d58e4d81db67bb7203
Link: https://chromium-review.googlesource.com/c/v8/v8/+/1103533

Use gn from PATH on aix
V8 Revision: e7fad930a83b5b4f1bfee6bee620f35b988d28c3
Link: https://chromium-review.googlesource.com/c/v8/v8/+/1168087

ppc64, aix: Pass CallFrequency object by const reference to avoid value copy error.
Bug: v8:8193
GCC bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61976
V8 Revision: d2e0166ded485df126c765b9196f7edd1ce50f82
Link: https://chromium-review.googlesource.com/c/v8/v8/+/1228633

disable failing cctest on AIX temporarily
V8 Revision: 67b549938d9060dde5cb557865f7451392a0f004
Link: https://chromium-review.googlesource.com/c/v8/v8/+/1174560

deps/v8/src/torque/file-visitor.h:
Add virtual dtor to avoid gcc error on Aix. (Not currently present in v8/master)

The following patch to v8/build solves this issue for v8/master:
a1a12ef3b343f9e75c630ed6dc8f1ea44a8a747b

However, the version of '/chromium/src/build.git' cannot be updated to include this
patch in v8/DEPS file. (could potentially cause issues for other platforms)

The change to deps/v8/src/torque/file-visitor.h is a workaround for origin/v10.x-staging
branch.

nodejs-github-bot added build Issues and PRs related to build files or the CI. v10.x v8 engine Issues and PRs related to the V8 dependency. labels Oct 16, 2018
V-for-Vasili mentioned this pull request Oct 16, 2018
Closed

V-for-Vasili commented Oct 16, 2018

Copy link
Copy Markdown
Contributor Author

cc @nodejs/v8

mmarchini commented Oct 16, 2018

Copy link
Copy Markdown
Contributor

/cc @nodejs/v8 @nodejs/v8-update

mhdawson commented Oct 16, 2018

Copy link
Copy Markdown
Member

I think you need to include e228033 in your description for the PR, but otherwise looks good.

mhdawson approved these changes Oct 16, 2018
View reviewed changes

mhdawson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

refack commented Oct 16, 2018

Copy link
Copy Markdown
Contributor

@V-for-Vasili if you want you can teach GitHub to recognize you by following https://help.github.com/articles/setting-your-commit-email-address-on-github/

V-for-Vasili commented Oct 17, 2018

Copy link
Copy Markdown
Contributor Author

@mhdawson
e228033 included in PR description:

Use gn from PATH on aix
V8 Revision: e7fad930a83b5b4f1bfee6bee620f35b988d28c3
Link: https://chromium-review.googlesource.com/c/v8/v8/+/1168087

BridgeAR commented Oct 17, 2018

Copy link
Copy Markdown
Member

CI https://ci.nodejs.org/job/node-test-pull-request/17945/

V-for-Vasili commented Oct 17, 2018

Copy link
Copy Markdown
Contributor Author

@BridgeAR according to the log a git exception (timeout) occurred for OSX1010
Would you be able to try and rerun the CI?

mhdawson commented Oct 23, 2018

Copy link
Copy Markdown
Member

Resume build: https://ci.nodejs.org/job/node-test-pull-request/18085/

mhdawson commented Oct 29, 2018

Copy link
Copy Markdown
Member

Resumed build again: https://ci.nodejs.org/job/node-test-pull-request/18215/

V-for-Vasili mentioned this pull request Oct 31, 2018
Closed

refack commented Oct 31, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor

Is there anything else I need to do for this

IIUC no. It just needs to be accepted by @nodejs/lts

mhdawson commented Oct 31, 2018

Copy link
Copy Markdown
Member

@BethGriggs can you see if we can pull this in for the next 8.x release.

mhdawson commented Oct 31, 2018

Copy link
Copy Markdown
Member

@BethGriggs sorry I meant the next 10.x release

BethGriggs commented Nov 1, 2018

Copy link
Copy Markdown
Member

Could someone from @nodejs/v8 review please?

MylesBorins commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

/cc @nodejs/v8-update

V-for-Vasili force-pushed the Node-10-aix-backport-cherry-pick branch 2 times, most recently from 8b3326b to 9e969c3 Compare November 14, 2018 16:12

V-for-Vasili commented Nov 14, 2018

Copy link
Copy Markdown
Contributor Author

@refack Can the common.gypi conflict be resolved when this PR is being merged using "Resolve conflicts" button or do I need to resolve it? (simple version mismatch in common.gypi: -node.36 vs -node.38)

refack commented Nov 14, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor

@refack Can the common.gypi conflict be resolved when this PR is being merged using "Resolve conflicts" button or do I need to resolve it? (simple version mismatch in common.gypi: -node.36 vs -node.38)

We don't use the "resolve conflicts" button because it creates a merge commit. The conflict could be solved during the manual landing process though. v8_embedder_string might also continue to change anyway while the the v10.x version is being staged.

MylesBorins previously requested changes Nov 21, 2018
View reviewed changes

MylesBorins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

common.gypi should be bumped for each individual commit. (currently it is being bumped once at the end of all the commits).

deps: cherry-pick d9e7832 from V8 upstream
895ac95 
Original commit message:
  fix gn builds on aix

  Change-Id: I60aed7bf8207703fa6ceddb6165e173e68b5ff5f
  Reviewed-on: https://chromium-review.googlesource.com/1103533
  Commit-Queue: Michael Achenbach <machenbach@chromium.org>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{nodejs#54386}
Vasili Skurydzin added 4 commits November 21, 2018 14:23
deps: cherry-pick a51f429 from V8 upstream:
997c075 
  Original commit message:
  Use gn from PATH on aix

  Change-Id: I853f7899dbba9112ba1ca2ce78e2838b5a09c975
  Reviewed-on: https://chromium-review.googlesource.com/1168087
  Commit-Queue: John Barboza <jbarboza@ca.ibm.com>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{nodejs#55028}
deps: cherry-pick d2e0166 from V8 upstream
6571084 
  Original commit message:

  ppc64, aix: Pass CallFrequency object by const reference to avoid value copy error.

  Bug: v8:8193
  GCC bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61976

  Change-Id: I0d4efca4da03ef82651325e15ddf2160022bc8de
  Reviewed-on: https://chromium-review.googlesource.com/1228633
  Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
  Reviewed-by: Daniel Clifford <danno@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{nodejs#56275}
deps: cherry-pick 67b5499 from V8 upstream
c6db967 
Original commit message:
  PPC: disable failing cctest on AIX temporarily

  Change-Id: I8a0081acb9c5eb662bf43eceb52218096eac327c
  Reviewed-on: https://chromium-review.googlesource.com/1174560
  Reviewed-by: Adam Klein <adamk@chromium.org>
  Reviewed-by: Camillo Bruni <cbruni@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{nodejs#55229}
deps/v8/src/torque/file-visitor.h:
21c50f1 
Add virtual dtor to avoid gcc error on Aix. (Not currently present in v8/master)

deps/v8/third_party/antlr4/BUILD.gn:
Use current_os variable to avoid is_aix being undefined.

The following patch to v8/build solves this issue for v8/master:
a1a12ef3b343f9e75c630ed6dc8f1ea44a8a747b

However, the version of '/chromium/src/build.git' cannot be updated to include this
patch in v8/DEPS file. (could potentially cause issues for other platforms)

The change to deps/v8/src/torque/file-visitor.h is a workaround for origin/v10.x-staging
branch.
V-for-Vasili force-pushed the Node-10-aix-backport-cherry-pick branch from 9e969c3 to 21c50f1 Compare November 21, 2018 21:10

V-for-Vasili commented Nov 21, 2018

Copy link
Copy Markdown
Contributor Author

@MylesBorins done.

MylesBorins commented Nov 22, 2018

Copy link
Copy Markdown
Contributor

CI: https://ci.nodejs.org/job/node-test-pull-request/18862/

MylesBorins pushed a commit that referenced this pull request Nov 22, 2018
deps: cherry-pick d9e7832 from V8 upstream
d4c8d77 
Original commit message:
  fix gn builds on aix

  Change-Id: I60aed7bf8207703fa6ceddb6165e173e68b5ff5f
  Reviewed-on: https://chromium-review.googlesource.com/1103533
  Commit-Queue: Michael Achenbach <machenbach@chromium.org>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#54386}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 22, 2018
deps: cherry-pick abab9fb from V8 upstream
01544e1 
Original commit message:
  s390, ppc64: Enable v8gen.py on Linux s390, ppc64

  Change-Id: Ia05e949e1a823e30a45894c47f6f6df2e159befe
  Reviewed-on: https://chromium-review.googlesource.com/1135540
  Commit-Queue: Michael Achenbach <machenbach@chromium.org>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#54485}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 22, 2018
deps: cherry-pick a51f429 from V8 upstream
a4d67c9 
Original commit message:

  Use gn from PATH on aix

  Change-Id: I853f7899dbba9112ba1ca2ce78e2838b5a09c975
  Reviewed-on: https://chromium-review.googlesource.com/1168087
  Commit-Queue: John Barboza <jbarboza@ca.ibm.com>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#55028}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 22, 2018
deps: cherry-pick d2e0166 from V8 upstream
f2d3bad 
Original commit message:

  ppc64, aix: Pass CallFrequency object by const reference to avoid value copy error.

  Bug: v8:8193
  GCC bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61976

  Change-Id: I0d4efca4da03ef82651325e15ddf2160022bc8de
  Reviewed-on: https://chromium-review.googlesource.com/1228633
  Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
  Reviewed-by: Daniel Clifford <danno@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{#56275}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 22, 2018
deps: cherry-pick 67b5499 from V8 upstream
d3fb599 
Original commit message:
  PPC: disable failing cctest on AIX temporarily

  Change-Id: I8a0081acb9c5eb662bf43eceb52218096eac327c
  Reviewed-on: https://chromium-review.googlesource.com/1174560
  Reviewed-by: Adam Klein <adamk@chromium.org>
  Reviewed-by: Camillo Bruni <cbruni@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{#55229}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 22, 2018
deps: V8: Add virtual dtor to avoid aix gcc error
20430ae 
deps/v8/src/torque/file-visitor.h:
* Not currently present in v8/master

deps/v8/third_party/antlr4/BUILD.gn:
* Use current_os variable to avoid is_aix being undefined.

The following patch to v8/build solves this issue for v8/master:
* a1a12ef3b343f9e75c630ed6dc8f1ea44a8a747b

However, the version of '/chromium/src/build.git' cannot be updated to
include this patch in v8/DEPS file. (could potentially cause issues for
other platforms)

The change to deps/v8/src/torque/file-visitor.h is a workaround for
origin/v10.x-staging branch.

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>

MylesBorins commented Nov 22, 2018

Copy link
Copy Markdown
Contributor

landed in 469473d...20430ae

codebytere mentioned this pull request Nov 27, 2018
Closed
rvagg pushed a commit that referenced this pull request Nov 28, 2018
deps: cherry-pick d9e7832 from V8 upstream
8d85275 
Original commit message:
  fix gn builds on aix

  Change-Id: I60aed7bf8207703fa6ceddb6165e173e68b5ff5f
  Reviewed-on: https://chromium-review.googlesource.com/1103533
  Commit-Queue: Michael Achenbach <machenbach@chromium.org>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#54386}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
rvagg pushed a commit that referenced this pull request Nov 28, 2018
deps: cherry-pick abab9fb from V8 upstream
51a8fe1 
Original commit message:
  s390, ppc64: Enable v8gen.py on Linux s390, ppc64

  Change-Id: Ia05e949e1a823e30a45894c47f6f6df2e159befe
  Reviewed-on: https://chromium-review.googlesource.com/1135540
  Commit-Queue: Michael Achenbach <machenbach@chromium.org>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#54485}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
rvagg pushed a commit that referenced this pull request Nov 28, 2018
deps: cherry-pick a51f429 from V8 upstream
a8b3b9d 
Original commit message:

  Use gn from PATH on aix

  Change-Id: I853f7899dbba9112ba1ca2ce78e2838b5a09c975
  Reviewed-on: https://chromium-review.googlesource.com/1168087
  Commit-Queue: John Barboza <jbarboza@ca.ibm.com>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#55028}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
rvagg pushed a commit that referenced this pull request Nov 28, 2018
deps: cherry-pick d2e0166 from V8 upstream
a7cdd67 
Original commit message:

  ppc64, aix: Pass CallFrequency object by const reference to avoid value copy error.

  Bug: v8:8193
  GCC bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61976

  Change-Id: I0d4efca4da03ef82651325e15ddf2160022bc8de
  Reviewed-on: https://chromium-review.googlesource.com/1228633
  Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
  Reviewed-by: Daniel Clifford <danno@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{#56275}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
rvagg pushed a commit that referenced this pull request Nov 28, 2018
deps: cherry-pick 67b5499 from V8 upstream
828ac19 
Original commit message:
  PPC: disable failing cctest on AIX temporarily

  Change-Id: I8a0081acb9c5eb662bf43eceb52218096eac327c
  Reviewed-on: https://chromium-review.googlesource.com/1174560
  Reviewed-by: Adam Klein <adamk@chromium.org>
  Reviewed-by: Camillo Bruni <cbruni@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{#55229}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
rvagg pushed a commit that referenced this pull request Nov 28, 2018
deps: V8: Add virtual dtor to avoid aix gcc error
729764a 
deps/v8/src/torque/file-visitor.h:
* Not currently present in v8/master

deps/v8/third_party/antlr4/BUILD.gn:
* Use current_os variable to avoid is_aix being undefined.

The following patch to v8/build solves this issue for v8/master:
* a1a12ef3b343f9e75c630ed6dc8f1ea44a8a747b

However, the version of '/chromium/src/build.git' cannot be updated to
include this patch in v8/DEPS file. (could potentially cause issues for
other platforms)

The change to deps/v8/src/torque/file-visitor.h is a workaround for
origin/v10.x-staging branch.

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 29, 2018
deps: cherry-pick d9e7832 from V8 upstream
e5f795a 
Original commit message:
  fix gn builds on aix

  Change-Id: I60aed7bf8207703fa6ceddb6165e173e68b5ff5f
  Reviewed-on: https://chromium-review.googlesource.com/1103533
  Commit-Queue: Michael Achenbach <machenbach@chromium.org>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#54386}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 29, 2018
deps: cherry-pick abab9fb from V8 upstream
e9901dd 
Original commit message:
  s390, ppc64: Enable v8gen.py on Linux s390, ppc64

  Change-Id: Ia05e949e1a823e30a45894c47f6f6df2e159befe
  Reviewed-on: https://chromium-review.googlesource.com/1135540
  Commit-Queue: Michael Achenbach <machenbach@chromium.org>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#54485}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 29, 2018
deps: cherry-pick a51f429 from V8 upstream
bb55cc1 
Original commit message:

  Use gn from PATH on aix

  Change-Id: I853f7899dbba9112ba1ca2ce78e2838b5a09c975
  Reviewed-on: https://chromium-review.googlesource.com/1168087
  Commit-Queue: John Barboza <jbarboza@ca.ibm.com>
  Reviewed-by: Michael Achenbach <machenbach@chromium.org>
  Cr-Commit-Position: refs/heads/master@{#55028}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 29, 2018
deps: cherry-pick d2e0166 from V8 upstream
4233bd9 
Original commit message:

  ppc64, aix: Pass CallFrequency object by const reference to avoid value copy error.

  Bug: v8:8193
  GCC bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61976

  Change-Id: I0d4efca4da03ef82651325e15ddf2160022bc8de
  Reviewed-on: https://chromium-review.googlesource.com/1228633
  Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
  Reviewed-by: Daniel Clifford <danno@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{#56275}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 29, 2018
deps: cherry-pick 67b5499 from V8 upstream
1bea8db 
Original commit message:
  PPC: disable failing cctest on AIX temporarily

  Change-Id: I8a0081acb9c5eb662bf43eceb52218096eac327c
  Reviewed-on: https://chromium-review.googlesource.com/1174560
  Reviewed-by: Adam Klein <adamk@chromium.org>
  Reviewed-by: Camillo Bruni <cbruni@chromium.org>
  Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
  Commit-Queue: Junliang Yan <jyan@ca.ibm.com>
  Cr-Commit-Position: refs/heads/master@{#55229}

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 29, 2018
deps: V8: Add virtual dtor to avoid aix gcc error
e231bf5 
deps/v8/src/torque/file-visitor.h:
* Not currently present in v8/master

deps/v8/third_party/antlr4/BUILD.gn:
* Use current_os variable to avoid is_aix being undefined.

The following patch to v8/build solves this issue for v8/master:
* a1a12ef3b343f9e75c630ed6dc8f1ea44a8a747b

However, the version of '/chromium/src/build.git' cannot be updated to
include this patch in v8/DEPS file. (could potentially cause issues for
other platforms)

The change to deps/v8/src/torque/file-visitor.h is a workaround for
origin/v10.x-staging branch.

PR-URL: #23695
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
codebytere mentioned this pull request Nov 29, 2018
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

5 more reviewers
refack refack approved these changes
hashseed hashseed approved these changes
mhdawson mhdawson approved these changes
ryzokuken ryzokuken approved these changes
MylesBorins MylesBorins left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

build Issues and PRs related to build files or the CI. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Back | FazBrowse Home | New Git URL