[test] Make cfi more verbose.

[test] Make cfi more verbose.

Set "no recover" in diagnostic mode to keep triggering
crashes. Otherwise, the v8 test driver won't notice tests
as failing.

Also port some common sanitizer cflags to cfi - those can be
removed on the build side afterwards.

BUG=chromium:515782
LOG=n

Committed: https://crrev.com/d6ebeb4e0d00d4f259ed48128ee1351775379a1c
Cr-Commit-Position: refs/heads/master@{#31442}

[Michael Achenbach, 2015-10-21 08:20:50]

Description was changed from

==========
[test] Make cfi more verbose.

BUG=
==========

to

==========
[test] Make cfi more verbose.

Set "no recover" in diagnostic mode to keep triggering
crashes. Otherwise, the v8 test driver won't notice tests
as failing.

Also port some common sanitizer cflags to cfi - those can be
removed on the build side afterwards.
==========

[Michael Achenbach, 2015-10-21 08:20:50]

machenbach@chromium.org changed reviewers:
+ glider@chromium.org, jochen@chromium.org

[Michael Achenbach, 2015-10-21 08:21:21]

Description was changed from

==========
[test] Make cfi more verbose.

Set "no recover" in diagnostic mode to keep triggering
crashes. Otherwise, the v8 test driver won't notice tests
as failing.

Also port some common sanitizer cflags to cfi - those can be
removed on the build side afterwards.
==========

to

==========
[test] Make cfi more verbose.

Set "no recover" in diagnostic mode to keep triggering
crashes. Otherwise, the v8 test driver won't notice tests
as failing.

Also port some common sanitizer cflags to cfi - those can be
removed on the build side afterwards.

BUG=chromium:515782
LOG=n
==========

[Michael Achenbach, 2015-10-21 08:21:55]

PTAL

[Michael Achenbach, 2015-10-21 08:21:59]

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

[commit-bot: I haz the power, 2015-10-21 08:22:06]

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

[commit-bot: I haz the power, 2015-10-21 08:49:40]

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

[commit-bot: I haz the power, 2015-10-21 08:49:40]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2015-10-21 13:21:30]

lgtm

[Michael Achenbach, 2015-10-21 13:30:03]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2015-10-21 13:30:06]

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

[Michael Achenbach, 2015-10-21 13:31:01]

The CQ bit was unchecked by machenbach@chromium.org

[Michael Achenbach, 2015-10-21 13:34:01]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2015-10-21 13:34:08]

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

[Alexander Potapenko, 2015-10-21 13:34:54]

LGTM

[commit-bot: I haz the power, 2015-10-21 13:35:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-21 13:35:53]

Patchset 2 (id:??) landed as
https://crrev.com/d6ebeb4e0d00d4f259ed48128ee1351775379a1c
Cr-Commit-Position: refs/heads/master@{#31442}