Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: v8/v8
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 6.5.254.31
Choose a base ref
...
head repository: v8/v8
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 6.5.254.38
Choose a head ref
  • 14 commits
  • 21 files changed
  • 7 contributors

Commits on Mar 8, 2018

  1. Merged: Revert "Reland: Reimplement Object.entries/values as CSA to o…

    …ptimize performance."
    
    This reverts commit 03e9d41.
    
    Reason for revert: Correctness issues, see https://crbug.com/804159.
    
    Bug: chromium:804159
    
    Original change's description:
    > Reland: Reimplement Object.entries/values as CSA to optimize performance.
    >
    > Add Object.entries/values builtins to debug-evaluate.cc whitelist macro.
    > This fix revert commit of https://chromium-review.googlesource.com/c/v8/v8/+/859937
    > Original is https://chromium-review.googlesource.com/c/v8/v8/+/810504
    > >> Reimplements Object.entries/values as CSA to optimize performance. See more detail about https://bugs.chromium.org/p/v8/issues/ Issue 6804.
    >
    > This reverts commit 1b49f72.
    >
    > Bug: v8:6804
    > Change-Id: I57e8b66e1c4ece2abb52e1630a97fbfd4070d810
    > Reviewed-on: https://chromium-review.googlesource.com/860679
    > Commit-Queue: Yang Guo <[email protected]>
    > Reviewed-by: Yang Guo <[email protected]>
    > Cr-Commit-Position: refs/heads/master@{#50492}
    
    [email protected],[email protected],[email protected],[email protected],[email protected]
    
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    
    Bug: v8:6804
    Change-Id: I39b1854ca7c2f57819ba377f84560356d3756bfb
    Reviewed-on: https://chromium-review.googlesource.com/877886
    Reviewed-by: Jakob Gruber <[email protected]>
    Commit-Queue: Jakob Gruber <[email protected]>
    Cr-Original-Commit-Position: refs/heads/master@{#50746}
    Reviewed-on: https://chromium-review.googlesource.com/955083
    Cr-Commit-Position: refs/branch-heads/6.5@{#63}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    schuay authored and Commit Bot committed Mar 8, 2018
    Configuration menu
    Copy the full SHA
    ee390ca View commit details
    Browse the repository at this point in the history
  2. Version 6.5.254.32

    [email protected]
    
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: Ia113aa4cc31999e3abaead5c68893f68944f4278
    Reviewed-on: https://chromium-review.googlesource.com/954925
    Reviewed-by: v8 autoroll <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#64}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    V8 Autoroll committed Mar 8, 2018
    Configuration menu
    Copy the full SHA
    71404a1 View commit details
    Browse the repository at this point in the history

Commits on Mar 12, 2018

  1. Merged: [runtime] Properly calculate upper bound for NOF in_object_pr…

    …operties
    
    Revision: 0d26307
    
    BUG=chromium:817499
    LOG=N
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    [email protected]
    
    Change-Id: I5ce7203243c0ac84d654a106131f20725833766a
    Reviewed-on: https://chromium-review.googlesource.com/958471
    Reviewed-by: Igor Sheludko <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#65}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    camillobruni committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    de1b8ea View commit details
    Browse the repository at this point in the history
  2. Version 6.5.254.33

    [email protected]
    
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: I83013274592fa418cdb165ef317801f27d16f04b
    Reviewed-on: https://chromium-review.googlesource.com/958211
    Reviewed-by: v8 autoroll <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#66}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    V8 Autoroll committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    d6976f7 View commit details
    Browse the repository at this point in the history
  3. Merged: [arm64] Fix compile error in Simulator

    Some compilers complain that ConvertReturn is private in SimulatorBase,
    but used in the arm64 Simulator. This CL fixes this by making
    ConvertReturn protected.
    
    [email protected]
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    
    Bug: v8:7541
    Change-Id: Ic36eeb6edbca59982714fe10e142cf2d1dbc5ecd
    Originally-reviewed-on: https://chromium-review.googlesource.com/955689
    Reviewed-on: https://chromium-review.googlesource.com/958901
    Reviewed-by: Andreas Haas <[email protected]>
    Commit-Queue: Clemens Hammacher <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#67}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    backes authored and Commit Bot committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    70e2a0b View commit details
    Browse the repository at this point in the history
  4. Version 6.5.254.34

    [email protected]
    
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: I3caa5cfbc1fdd0ac0aee188f154e6c9f66f843f0
    Reviewed-on: https://chromium-review.googlesource.com/958214
    Reviewed-by: v8 autoroll <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#68}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    V8 Autoroll committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    333e637 View commit details
    Browse the repository at this point in the history
  5. Merged: [heap] Make setting use_tasks_ explicit

    At the moment the flag is set too late, it is possible that the GC still
    tries to post tasks in Isolate::Deinit when the isolate is already
    disconnected from the platform, see the referenced bug.
    
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    R=​[email protected]
    
    Bug: chromium:810739
    Change-Id: Ibcd226cb44cc903f2a46e7cccf682b3938c9d408
    Reviewed-on: https://chromium-review.googlesource.com/915942
    Reviewed-by: Ulan Degenbaev <[email protected]>
    Commit-Queue: Andreas Haas <[email protected]>
    Cr-Original-Commit-Position: refs/heads/master@{#51274}(cherry picked from commit 1e9504b)
    Reviewed-on: https://chromium-review.googlesource.com/958961
    Cr-Commit-Position: refs/branch-heads/6.5@{#69}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    gahaas authored and Commit Bot committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    c2a6be0 View commit details
    Browse the repository at this point in the history
  6. Version 6.5.254.35

    [email protected]
    
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: Id94a7697b1b6ca9cbf3f21a69e9e09cc4b08a401
    Reviewed-on: https://chromium-review.googlesource.com/958216
    Reviewed-by: v8 autoroll <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#70}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    V8 Autoroll committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    4f7a5e4 View commit details
    Browse the repository at this point in the history
  7. Revert "Merged: [heap] Make setting use_tasks_ explicit"

    This reverts commit c2a6be0.
    
    Reason for revert: A second CL is needed to fix this issue, and the second CL is not ready yet for backmerge.
    
    Original change's description:
    > Merged: [heap] Make setting use_tasks_ explicit
    > 
    > At the moment the flag is set too late, it is possible that the GC still
    > tries to post tasks in Isolate::Deinit when the isolate is already
    > disconnected from the platform, see the referenced bug.
    > 
    > NOTRY=true
    > NOPRESUBMIT=true
    > NOTREECHECKS=true
    > R=​​[email protected]
    > 
    > Bug: chromium:810739
    > Change-Id: Ibcd226cb44cc903f2a46e7cccf682b3938c9d408
    > Reviewed-on: https://chromium-review.googlesource.com/915942
    > Reviewed-by: Ulan Degenbaev <[email protected]>
    > Commit-Queue: Andreas Haas <[email protected]>
    > Cr-Original-Commit-Position: refs/heads/master@{#51274}(cherry picked from commit 1e9504b)
    > Reviewed-on: https://chromium-review.googlesource.com/958961
    > Cr-Commit-Position: refs/branch-heads/6.5@{#69}
    > Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    > Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    
    [email protected],[email protected]
    
    Change-Id: I12ffd87eb2242ff33c6f83cb28349dfcd8a069d1
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:810739
    Reviewed-on: https://chromium-review.googlesource.com/958910
    Reviewed-by: Andreas Haas <[email protected]>
    Commit-Queue: Andreas Haas <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#71}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    gahaas authored and Commit Bot committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    d26cccf View commit details
    Browse the repository at this point in the history
  8. Version 6.5.254.36

    [email protected]
    
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: Iec375d017da929438e8babbacbbf308357437af8
    Reviewed-on: https://chromium-review.googlesource.com/957912
    Reviewed-by: v8 autoroll <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#72}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    V8 Autoroll committed Mar 12, 2018
    Configuration menu
    Copy the full SHA
    d027d08 View commit details
    Browse the repository at this point in the history

Commits on Mar 13, 2018

  1. Merged: Squashed multiple commits.

    Merged: [profiler] use Sleep() on windows for long profile intervals.
    Revision: 15c0c3a
    
    Merged: [profiler] fix sleeping on windows for long intervals.
    Revision: 43e2fb1
    
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    [email protected]
    
    Bug: v8:7535
    Change-Id: If426b1282e4b4c0ab765624f681415fe943f5170
    Reviewed-on: https://chromium-review.googlesource.com/960845
    Reviewed-by: Yang Guo <[email protected]>
    Commit-Queue: Ali Ijaz Sheikh <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#73}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    ofrobots authored and Commit Bot committed Mar 13, 2018
    Configuration menu
    Copy the full SHA
    0ad46e7 View commit details
    Browse the repository at this point in the history
  2. Version 6.5.254.37

    [email protected]
    
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: If4d20a99a543d20963b0443364bf23c3e913181b
    Reviewed-on: https://chromium-review.googlesource.com/960923
    Reviewed-by: v8 autoroll <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#74}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    V8 Autoroll committed Mar 13, 2018
    Configuration menu
    Copy the full SHA
    8dc45f6 View commit details
    Browse the repository at this point in the history
  3. [inspector] Make test byte order independent

    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    
    Change-Id: If0fdc76170ad29b4d3dadddbb32bc87c307c04af
    Reviewed-on: https://chromium-review.googlesource.com/881883
    Reviewed-by: Aleksey Kozyatinskiy <[email protected]>
    Commit-Queue: Eugene Ostroukhov <[email protected]>
    Cr-Original-Commit-Position: refs/heads/master@{#50817}
    Bug: v8:7526
    Reviewed-on: https://chromium-review.googlesource.com/957789
    Cr-Commit-Position: refs/branch-heads/6.5@{#75}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    Eugene Ostroukhov authored and Commit Bot committed Mar 13, 2018
    Configuration menu
    Copy the full SHA
    6fc10d9 View commit details
    Browse the repository at this point in the history
  4. Version 6.5.254.38

    [email protected]
    
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: Ica3e74fbfce5acac567be3443ce9c7fe116c4e63
    Reviewed-on: https://chromium-review.googlesource.com/961092
    Reviewed-by: v8 autoroll <[email protected]>
    Cr-Commit-Position: refs/branch-heads/6.5@{#76}
    Cr-Branched-From: 73c55f5-refs/heads/6.5.254@{#1}
    Cr-Branched-From: 594a1a0-refs/heads/master@{#50664}
    V8 Autoroll committed Mar 13, 2018
    Configuration menu
    Copy the full SHA
    9d37823 View commit details
    Browse the repository at this point in the history
Loading