[v8-dev] Reland II of 'Optimize trivial regexp disjunctions' CL 1176453002 (issue 1180433003 by erikco...@chromium.org)

2015-06-10 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Yang, Description: Reland II of 'Optimize trivial regexp disjunctions' CL 1176453002 R=yang...@chromium.org BUG= Please review this at https://codereview.chromium.org/1180433003/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+251, -8 lines): M src/a

[v8-dev] Reland of 'Optimize trivial regexp disjunctions' CL 1176453002 (issue 1174713002 by erikco...@chromium.org)

2015-06-09 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Yang, Description: Reland of 'Optimize trivial regexp disjunctions' CL 1176453002 Original code review: https://codereview.chromium.org/1176453002/ TBR=yang...@chromium.org BUG=chromium:482998 LOG=n Please review this at https://codereview.chromium.org/1174713002/ Base URL: https:/

[v8-dev] Optimize trivial regexp disjunctions (issue 1176453002 by erikco...@chromium.org)

2015-06-09 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Yang, Description: Optimize trivial regexp disjunctions R=yang...@chromium.org BUG=chromium:482998 LOG=n Please review this at https://codereview.chromium.org/1176453002/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+235, -4 lines): M src/ast.h M

[v8-dev] Tiny fix to grokdump heap stats printer (issue 1164693002 by erikco...@chromium.org)

2015-06-01 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Hannes Payer, Description: Tiny fix to grokdump heap stats printer R=hpa...@chromium.org BUG= Please review this at https://codereview.chromium.org/1164693002/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+1, -1 lines): M tools/grokdump.py Index:

[v8-dev] Cosmetic changes to tests to make it easier to concatenate them. (issue 1163803002 by erikco...@chromium.org)

2015-06-01 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: ulan, Description: Cosmetic changes to tests to make it easier to concatenate them. When compiling on a laptop I like to concatenate the small test files. This makes a big difference to compile times. These changes make that easier. R=u...@chromium.org BUG= Please review this at htt

[v8-dev] Remove spurious prints from GC logging (issue 1149413003 by erikco...@chromium.org)

2015-05-30 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Hannes Payer, szager1, Description: Remove spurious prints from GC logging R=hpa...@chromium.org, sza...@chromium.org BUG= Please review this at https://codereview.chromium.org/1149413003/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+1, -1 lines):

[v8-dev] Even without --trace-gc dump the last few GC messages on OOM (issue 1159513003 by erikco...@chromium.org)

2015-05-29 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Hannes Payer, Description: Even without --trace-gc dump the last few GC messages on OOM If we crash V8 due to out-of-memory then we print the last 3 GCs on stdout as we crash. Also records the last 3 GCs on the stack so that it will be part of the minidump. R=hpa...@chromium.org BUG=

[v8-dev] grokdump.py - some support for on-stack HeapStats (issue 1142343009 by erikco...@chromium.org)

2015-05-28 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: ulan, Description: grokdump.py - some support for on-stack HeapStats R=u...@chromium.org BUG= Please review this at https://codereview.chromium.org/1142343009/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+22, -4 lines): M tools/grokdump.py Index

[v8-dev] Re: Version 4.3.61.27 (cherry-pick) (issue 1163563002 by erikco...@chromium.org)

2015-05-28 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Committed patchset #1 (id:1) manually as 85e92361bb42b9076ebdcb1c005fce9242bc5fe6. https://codereview.chromium.org/1163563002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v

[v8-dev] Version 4.3.61.27 (cherry-pick) (issue 1163563002 by erikco...@chromium.org)

2015-05-28 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Hannes Payer, Description: Version 4.3.61.27 (cherry-pick) Merged 49fdc5323 Be sure to abort incremental GC when trying to reserve space R=hpa...@chromium.org BUG= Please review this at https://codereview.chromium.org/1163563002/ Base URL: https://chromium.googlesource.com/v8/v8.g

[v8-dev] Re: Version 4.3.61.26 (cherry-pick) (issue 1157393002 by erikco...@chromium.org)

2015-05-28 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Committed patchset #1 (id:1) manually as 17db4ce193547f334c6cc63e27e1f9a0b434fcb7. https://codereview.chromium.org/1157393002/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v

[v8-dev] Version 4.3.61.26 (cherry-pick) (issue 1157393002 by erikco...@chromium.org)

2015-05-28 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Yang, Description: Version 4.3.61.26 (cherry-pick) Merged 5a93a3304 Reland: Fix JSON parser Handle leak (previous CL 1041483004) BUG=472504,v8:3976 LOG=N R=yang...@chromium.org Please review this at https://codereview.chromium.org/1157393002/ Base URL: https://chromium.googlesourc

[v8-dev] Version 4.3.61.26 (cherry-pick) (issue 1159063003 by erikco...@chromium.org)

2015-05-28 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Yang, Description: Version 4.3.61.26 (cherry-pick) Merged 5a93a3304 Reland: Fix JSON parser Handle leak (previous CL 1041483004) BUG=472504,v8:3976 LOG=N R=yang...@chromium.org Please review this at https://codereview.chromium.org/1159063003/ Base URL: https://chromium.googlesourc

[v8-dev] grokdump.py: work around int size limits on xrange (issue 1158933007 by erikco...@chromium.org)

2015-05-28 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Hannes Payer, Description: grokdump.py: work around int size limits on xrange R=hpa...@chromium.org BUG= Please review this at https://codereview.chromium.org/1158933007/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+23, -15 lines): M tools/grokdum

[v8-dev] Random hashes with a more cache-friendly distribution (issue 1158083002 by erikco...@chromium.org)

2015-05-26 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: adamk, Toon Verwaest, Description: Random hashes with a more cache-friendly distribution R=ad...@chromium.org, verwa...@chromium.org BUG= Please review this at https://codereview.chromium.org/1158083002/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+

[v8-dev] Speed up object-keyed Map and Set by giving out sequential hash codes (issue 1157073002 by erikco...@chromium.org)

2015-05-26 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: adamk, Toon Verwaest, Description: Speed up object-keyed Map and Set by giving out sequential hash codes R=ad...@chromium.org, verwa...@chromium.org BUG= Please review this at https://codereview.chromium.org/1157073002/ Base URL: https://chromium.googlesource.com/v8/v8.git@master A

[v8-dev] Move hash code from hidden string to a private symbol (issue 1149863005 by erikco...@chromium.org)

2015-05-21 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: adamk, toon_chromium.org, Description: Move hash code from hidden string to a private symbol R=ad...@chromium.org, t...@chromium.org BUG= Please review this at https://codereview.chromium.org/1149863005/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+

[v8-dev] Regularize namespace closing curlies (issue 1143133002 by erikco...@chromium.org)

2015-05-20 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Toon Verwaest, Description: Regularize namespace closing curlies R=verwa...@chromium.org BUG= Please review this at https://codereview.chromium.org/1143133002/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+67, -67 lines): M src/compiler/ast-loop-as

[v8-dev] Adjust benchmark framework to avoid spending 50% of time on 'new Date' (issue 1133843007 by erikco...@chromium.org)

2015-05-19 Thread 'Erik Corry Chromium.org' via codereview.chromium.org
Reviewers: Dmitry Lomov (chromium), Description: Adjust benchmark framework to avoid spending 50% of time on 'new Date' R=dslo...@chromium.org BUG= Please review this at https://codereview.chromium.org/1133843007/ Base URL: https://chromium.googlesource.com/v8/v8.git@master Affected files (+6