[regexp] add fuzzer support for regexp parser and compiler.

[regexp] add fuzzer support for regexp parser and compiler.

R=jochen@chromium.org
BUG=chromium:577261
LOG=N

Committed: https://crrev.com/9f5dca8faa05d70b89ad9b795b0feed1513615aa
Cr-Commit-Position: refs/heads/master@{#33640}

[Yang, 2016-02-01 11:59:17]

On 2016/02/01 11:57:51, Yang wrote:

The test cases are taken from test/cctest/test-regexp.cc

[Yang, 2016-02-01 12:00:18]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-01 12:00:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1655853002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1655853002/1

[jochen (gone - plz use gerrit), 2016-02-01 12:03:06]

lgtm

[commit-bot: I haz the power, 2016-02-01 12:06:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-01 12:06:31]

Dry run: Try jobs failed on following builders:
  v8_mac_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel/builds/14791)

[Yang, 2016-02-01 12:08:04]

Description was changed from

==========
[regexp] add fuzzer support for regexp parser and compiler.

R=jochen@chromium.org
==========

to

==========
[regexp] add fuzzer support for regexp parser and compiler.

R=jochen@chromium.org
BUG=chromium:577261
LOG=N
==========

[Yang, 2016-02-01 12:15:35]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-02-01 12:15:35]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1655853002/#ps20001
(title: "fix swarming")

[commit-bot: I haz the power, 2016-02-01 12:15:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1655853002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1655853002/20001

[Yang, 2016-02-01 12:26:10]

The CQ bit was unchecked by yangguo@chromium.org

[Yang, 2016-02-01 12:36:58]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-02-01 12:36:58]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1655853002/#ps40001
(title: "also test two-byte")

[commit-bot: I haz the power, 2016-02-01 12:37:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1655853002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1655853002/40001

[Michael Achenbach, 2016-02-01 12:48:30]

The CQ bit was unchecked by machenbach@chromium.org

[Michael Achenbach, 2016-02-01 12:49:46]

I unchecked for now. Not liking this output:
https://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng_triggered/...

Especially from an infra pov. I'll have to look into why the runner isn't going
properly red. I'll wait for the output of the latest patch...

[Yang, 2016-02-01 12:51:22]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-01 12:51:27]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1655853002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1655853002/60001

[Michael Achenbach, 2016-02-01 12:51:55]

machenbach@chromium.org changed reviewers:
+ machenbach@chromium.org

[Michael Achenbach, 2016-02-01 12:51:56]

https://codereview.chromium.org/1655853002/diff/20001/test/fuzzer/fuzzer.isolate
File test/fuzzer/fuzzer.isolate (right):

https://codereview.chromium.org/1655853002/diff/20001/test/fuzzer/fuzzer.isol...
test/fuzzer/fuzzer.isolate:8:
'<(PRODUCT_DIR)/parser_fuzzer<(EXECUTABLE_SUFFIX)',
To fix the problems of this patch you also need here:
'<(PRODUCT_DIR)/regexp_fuzzer<(EXECUTABLE_SUFFIX)',

[Yang, 2016-02-01 12:52:16]

The CQ bit was unchecked by yangguo@chromium.org

[Michael Achenbach, 2016-02-01 12:52:25]

The CQ bit was unchecked by machenbach@chromium.org

[Yang, 2016-02-01 12:53:10]

On 2016/02/01 12:51:56, Michael Achenbach wrote:
>
https://codereview.chromium.org/1655853002/diff/20001/test/fuzzer/fuzzer.isolate
> File test/fuzzer/fuzzer.isolate (right):
> 
>
https://codereview.chromium.org/1655853002/diff/20001/test/fuzzer/fuzzer.isol...
> test/fuzzer/fuzzer.isolate:8:
> '<(PRODUCT_DIR)/parser_fuzzer<(EXECUTABLE_SUFFIX)',
> To fix the problems of this patch you also need here:
> '<(PRODUCT_DIR)/regexp_fuzzer<(EXECUTABLE_SUFFIX)',

Done. Thanks!

[Yang, 2016-02-01 12:53:15]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-01 12:53:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1655853002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1655853002/80001

[commit-bot: I haz the power, 2016-02-01 13:00:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-01 13:00:55]

Dry run: Try jobs failed on following builders:
  v8_win_compile_dbg on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_compile_dbg/builds/1...)

[jochen (gone - plz use gerrit), 2016-02-01 13:11:03]

https://codereview.chromium.org/1655853002/diff/80001/test/fuzzer/fuzzer.gyp
File test/fuzzer/fuzzer.gyp (right):

https://codereview.chromium.org/1655853002/diff/80001/test/fuzzer/fuzzer.gyp#...
test/fuzzer/fuzzer.gyp:94: 'parser_fuzzer',
probably also need to add regexp_fuzzer here?

[Yang, 2016-02-01 13:13:00]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-01 13:13:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1655853002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1655853002/100001

[Yang, 2016-02-01 13:13:10]

On 2016/02/01 13:11:03, jochen wrote:
> https://codereview.chromium.org/1655853002/diff/80001/test/fuzzer/fuzzer.gyp
> File test/fuzzer/fuzzer.gyp (right):
> 
>
https://codereview.chromium.org/1655853002/diff/80001/test/fuzzer/fuzzer.gyp#...
> test/fuzzer/fuzzer.gyp:94: 'parser_fuzzer',
> probably also need to add regexp_fuzzer here?

thanks. done.

[Yang, 2016-02-01 13:54:13]

The CQ bit was unchecked by yangguo@chromium.org

[Yang, 2016-02-01 13:54:14]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-02-01 13:54:14]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1655853002/#ps100001
(title: "fix fuzzer.gyp")

[commit-bot: I haz the power, 2016-02-01 13:54:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1655853002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1655853002/100001

[commit-bot: I haz the power, 2016-02-01 14:00:19]

Description was changed from

==========
[regexp] add fuzzer support for regexp parser and compiler.

R=jochen@chromium.org
BUG=chromium:577261
LOG=N
==========

to

==========
[regexp] add fuzzer support for regexp parser and compiler.

R=jochen@chromium.org
BUG=chromium:577261
LOG=N
==========

[commit-bot: I haz the power, 2016-02-01 14:00:20]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-02-01 14:00:45]

Description was changed from

==========
[regexp] add fuzzer support for regexp parser and compiler.

R=jochen@chromium.org
BUG=chromium:577261
LOG=N
==========

to

==========
[regexp] add fuzzer support for regexp parser and compiler.

R=jochen@chromium.org
BUG=chromium:577261
LOG=N

Committed: https://crrev.com/9f5dca8faa05d70b89ad9b795b0feed1513615aa
Cr-Commit-Position: refs/heads/master@{#33640}
==========

[commit-bot: I haz the power, 2016-02-01 14:00:46]

Patchset 6 (id:??) landed as
https://crrev.com/9f5dca8faa05d70b89ad9b795b0feed1513615aa
Cr-Commit-Position: refs/heads/master@{#33640}

[kcc2, 2016-02-06 04:00:54]

On 2016/02/01 14:00:46, commit-bot: I haz the power wrote:
> Patchset 6 (id:??) landed as
> https://crrev.com/9f5dca8faa05d70b89ad9b795b0feed1513615aa
> Cr-Commit-Position: refs/heads/master@{#33640}

btw, the fuzzer is *very* slow, about 2 exec/s

[kcc2, 2016-02-06 04:00:57]

On 2016/02/01 14:00:46, commit-bot: I haz the power wrote:
> Patchset 6 (id:??) landed as
> https://crrev.com/9f5dca8faa05d70b89ad9b795b0feed1513615aa
> Cr-Commit-Position: refs/heads/master@{#33640}

btw, the fuzzer is *very* slow, about 2 exec/s