1. 26 Mar, 2018 1 commit
  2. 22 Mar, 2018 3 commits
  3. 20 Mar, 2018 2 commits
  4. 19 Mar, 2018 2 commits
  5. 16 Mar, 2018 1 commit
  6. 15 Mar, 2018 1 commit
  7. 14 Mar, 2018 1 commit
    • Caitlin Potter's avatar
      Reland "[esnext] re-implement template strings" · b8229612
      Caitlin Potter authored
      - Add a new bytecode for the ToString operation, replacing the old
      intrinsic call (currently does not collect type feedback).
      - Add a new AST node to represent TemplateLiterals, and avoid
      generating unnecessary ToString operations in some simple cases.
      - Use a single feedback slot for each string addition, because the
      type feedback should always be the same for each addition
      
      This seems to produce a very slight improvement on JSTests benchmarks
      and bench-ruben.js from v8:7415, and it's possible that type feedback
      for the ToString bytecode could provide more opportunities to eliminate
      the runtime call in TurboFan.
      
      Doesn't touch tagged templates
      
      [esnext] fix OOB read in ASTPrinter::VisistTemplateLiteral
      
      Fixes an error where TemplateLiteral printing in --print-ast
      would try to read an element beyond the length of a vector.
      
      BUG=v8:7415, chromium:820596
      R=adamk@chromium.org, gsathya@chromum.org, rmcilroy@chromium.org, ishell@chromium.org, bmeurer@chromium.org
      
      Change-Id: Ie56894f73a6445550a5f95f42160c4e29ab1da42
      Reviewed-on: https://chromium-review.googlesource.com/958408Reviewed-by: 's avatarBenedikt Meurer <bmeurer@chromium.org>
      Commit-Queue: Caitlin Potter <caitp@igalia.com>
      Cr-Commit-Position: refs/heads/master@{#51933}
      b8229612
  8. 12 Mar, 2018 1 commit
  9. 10 Mar, 2018 1 commit
    • Michael Achenbach's avatar
      Revert "[esnext] re-implement template strings" · b8128279
      Michael Achenbach authored
      This reverts commit 8ae19e08.
      
      Reason for revert:
      Speculative revert for layout test:
      https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/22215
      
      See:
      https://github.com/v8/v8/wiki/Blink-layout-tests
      
      Original change's description:
      > [esnext] re-implement template strings
      > 
      > - Add a new bytecode for the ToString operation, replacing the old
      > intrinsic call (currently does not collect type feedback).
      > - Add a new AST node to represent TemplateLiterals, and avoid
      > generating unnecessary ToString operations in some simple cases.
      > - Use a single feedback slot for each string addition, because the
      > type feedback should always be the same for each addition
      > 
      > This seems to produce a very slight improvement on JSTests benchmarks
      > and bench-ruben.js from v8:7415, and it's possible that type feedback
      > for the ToString bytecode could provide more opportunities to eliminate
      > the runtime call in TurboFan.
      > 
      > Doesn't touch tagged templates
      > 
      > BUG=v8:7415
      > R=​rmcilroy@chromium.org, ishell@chromium.org, bmeurer@chromium.org
      > 
      > Change-Id: If5a8c68558431f058db894d65776324abf54218e
      > Reviewed-on: https://chromium-review.googlesource.com/945408
      > Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
      > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org>
      > Commit-Queue: Caitlin Potter <caitp@igalia.com>
      > Cr-Commit-Position: refs/heads/master@{#51853}
      
      TBR=rmcilroy@chromium.org,caitp@igalia.com,ishell@chromium.org,bmeurer@chromium.org
      
      Change-Id: Id0529b065493ffc20c8f2b1abacc4c1484c3c046
      No-Presubmit: true
      No-Tree-Checks: true
      No-Try: true
      Bug: v8:7415
      Reviewed-on: https://chromium-review.googlesource.com/958163Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
      Commit-Queue: Michael Achenbach <machenbach@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#51862}
      b8128279
  10. 09 Mar, 2018 2 commits
    • Caitlin Potter's avatar
      [esnext] re-implement template strings · 8ae19e08
      Caitlin Potter authored
      - Add a new bytecode for the ToString operation, replacing the old
      intrinsic call (currently does not collect type feedback).
      - Add a new AST node to represent TemplateLiterals, and avoid
      generating unnecessary ToString operations in some simple cases.
      - Use a single feedback slot for each string addition, because the
      type feedback should always be the same for each addition
      
      This seems to produce a very slight improvement on JSTests benchmarks
      and bench-ruben.js from v8:7415, and it's possible that type feedback
      for the ToString bytecode could provide more opportunities to eliminate
      the runtime call in TurboFan.
      
      Doesn't touch tagged templates
      
      BUG=v8:7415
      R=rmcilroy@chromium.org, ishell@chromium.org, bmeurer@chromium.org
      
      Change-Id: If5a8c68558431f058db894d65776324abf54218e
      Reviewed-on: https://chromium-review.googlesource.com/945408Reviewed-by: 's avatarBenedikt Meurer <bmeurer@chromium.org>
      Reviewed-by: 's avatarRoss McIlroy <rmcilroy@chromium.org>
      Commit-Queue: Caitlin Potter <caitp@igalia.com>
      Cr-Commit-Position: refs/heads/master@{#51853}
      8ae19e08
    • Mathias Bynens's avatar
      [builtins] Refactor the TypedArrayConstructor builtin · b10dc352
      Mathias Bynens authored
      This patch removes the TypedArrayConstructor_ConstructStub builtin,
      merging its functionality into the refactored TypedArrayConstructor
      TurboFan builtin.
      
      This brings us closer to our goal of deprecating the `construct_stub`
      field in `SharedFunctionInfo`.
      
      Bug: v8:7503, v8:7518
      Change-Id: I264aa0933b3c80814e021b05033636196dc289da
      Reviewed-on: https://chromium-review.googlesource.com/956044
      Commit-Queue: Mathias Bynens <mathias@chromium.org>
      Reviewed-by: 's avatarBenedikt Meurer <bmeurer@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#51843}
      b10dc352
  11. 06 Mar, 2018 1 commit
  12. 28 Feb, 2018 1 commit
  13. 23 Feb, 2018 3 commits
  14. 22 Feb, 2018 3 commits
  15. 21 Feb, 2018 1 commit
  16. 20 Feb, 2018 1 commit
  17. 16 Feb, 2018 1 commit
  18. 15 Feb, 2018 1 commit
  19. 08 Feb, 2018 1 commit
  20. 07 Feb, 2018 1 commit
  21. 30 Jan, 2018 1 commit
  22. 22 Jan, 2018 1 commit
  23. 19 Jan, 2018 1 commit
  24. 17 Jan, 2018 2 commits
  25. 12 Jan, 2018 1 commit
  26. 11 Jan, 2018 1 commit
  27. 02 Jan, 2018 3 commits
    • Bill Budge's avatar
      Revert "Revert "[debugger] do not try to find saved context before DebugEvaluate::Local"" · 781fa726
      Bill Budge authored
      This reverts commit 2a8e1e4a.
      
      Reason for revert: Time outs were caused by infra issues.
      
      Original change's description:
      > Revert "[debugger] do not try to find saved context before DebugEvaluate::Local"
      > 
      > This reverts commit ea7499df.
      > 
      > Reason for revert: Speculative, but this CL seems correlated with a bunch of timeouts on Linux:
      > 
      > https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap/builds/19540
      > 
      > Original change's description:
      > > [debugger] do not try to find saved context before DebugEvaluate::Local
      > > 
      > > Current implementation produces crashes since sometimes saved context
      > > is empty. It looks like we do not need to restore saved context since
      > > we do not set debug context as current, at least all tests are passed.
      > > 
      > > R=​yangguo@chromium.org
      > > 
      > > Bug: chromium:797573,chromium:792838
      > > Change-Id: I1271640f6a18cbaaecfa1e99ed9ac28e0dbbb1da
      > > Reviewed-on: https://chromium-review.googlesource.com/844979
      > > Reviewed-by: Yang Guo <yangguo@chromium.org>
      > > Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
      > > Cr-Commit-Position: refs/heads/master@{#50327}
      > 
      > TBR=yangguo@chromium.org,kozyatinskiy@chromium.org
      > 
      > Change-Id: I543201698c96c9762c481c1f6012cc13cb712842
      > No-Presubmit: true
      > No-Tree-Checks: true
      > No-Try: true
      > Bug: chromium:797573, chromium:792838
      > Reviewed-on: https://chromium-review.googlesource.com/846205
      > Reviewed-by: Bill Budge <bbudge@chromium.org>
      > Commit-Queue: Bill Budge <bbudge@chromium.org>
      > Cr-Commit-Position: refs/heads/master@{#50331}
      
      TBR=bbudge@chromium.org,yangguo@chromium.org,kozyatinskiy@chromium.org
      
      Change-Id: I44e916c796b935c372ce418a3fad44aa25d6f863
      No-Presubmit: true
      No-Tree-Checks: true
      No-Try: true
      Bug: chromium:797573, chromium:792838
      Reviewed-on: https://chromium-review.googlesource.com/848132Reviewed-by: 's avatarBill Budge <bbudge@chromium.org>
      Commit-Queue: Bill Budge <bbudge@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#50332}
      781fa726
    • Bill Budge's avatar
      Revert "[debugger] do not try to find saved context before DebugEvaluate::Local" · 2a8e1e4a
      Bill Budge authored
      This reverts commit ea7499df.
      
      Reason for revert: Speculative, but this CL seems correlated with a bunch of timeouts on Linux:
      
      https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap/builds/19540
      
      Original change's description:
      > [debugger] do not try to find saved context before DebugEvaluate::Local
      > 
      > Current implementation produces crashes since sometimes saved context
      > is empty. It looks like we do not need to restore saved context since
      > we do not set debug context as current, at least all tests are passed.
      > 
      > R=​yangguo@chromium.org
      > 
      > Bug: chromium:797573,chromium:792838
      > Change-Id: I1271640f6a18cbaaecfa1e99ed9ac28e0dbbb1da
      > Reviewed-on: https://chromium-review.googlesource.com/844979
      > Reviewed-by: Yang Guo <yangguo@chromium.org>
      > Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
      > Cr-Commit-Position: refs/heads/master@{#50327}
      
      TBR=yangguo@chromium.org,kozyatinskiy@chromium.org
      
      Change-Id: I543201698c96c9762c481c1f6012cc13cb712842
      No-Presubmit: true
      No-Tree-Checks: true
      No-Try: true
      Bug: chromium:797573, chromium:792838
      Reviewed-on: https://chromium-review.googlesource.com/846205Reviewed-by: 's avatarBill Budge <bbudge@chromium.org>
      Commit-Queue: Bill Budge <bbudge@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#50331}
      2a8e1e4a
    • Alexey Kozyatinskiy's avatar
      [debugger] do not try to find saved context before DebugEvaluate::Local · ea7499df
      Alexey Kozyatinskiy authored
      Current implementation produces crashes since sometimes saved context
      is empty. It looks like we do not need to restore saved context since
      we do not set debug context as current, at least all tests are passed.
      
      R=yangguo@chromium.org
      
      Bug: chromium:797573,chromium:792838
      Change-Id: I1271640f6a18cbaaecfa1e99ed9ac28e0dbbb1da
      Reviewed-on: https://chromium-review.googlesource.com/844979Reviewed-by: 's avatarYang Guo <yangguo@chromium.org>
      Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#50327}
      ea7499df
  28. 05 Dec, 2017 1 commit