[libfuzzer] Add "Symbolization" and "Debugging" sections to reproducing.md page.

[libfuzzer] Add "Symbolization" and "Debugging" sections to reproducing.md page.

R=aizatsky@chromium.org, inferno@chromium.org, ochang@chromium.org, thakis@chromium.org
BUG=640967
Committed: https://crrev.com/03e2e556db0bd18809f75059d0fc1804324d0e82
Cr-Commit-Position: refs/heads/master@{#415014}

[Nico, 2016-08-29 12:04:13]

Thanks, this is very helpful.

https://codereview.chromium.org/2288753002/diff/1/testing/libfuzzer/reproduci...
File testing/libfuzzer/reproducing.md (right):

https://codereview.chromium.org/2288753002/diff/1/testing/libfuzzer/reproduci...
testing/libfuzzer/reproducing.md:93: make sure `llvm-symbolizer` is in `PATH` or
provide it in separate
Mention that llvm-symbolizer is in third_party/llvm-build/Release+Asserts/bin
and/or that it's copied to the build dir at build time (and make the example
point there)

https://codereview.chromium.org/2288753002/diff/1/testing/libfuzzer/reproduci...
testing/libfuzzer/reproducing.md:109: Please checkout
[AddressSanitizerAndDebugger] page for some tips on debugging of
s/checkout/look at/ since "checkout" sounds like `svn checkout`. (Also,
"checkout" is the noun, while the verb is "check out" with a space).

Doesn't abort_on_error work in ASAN_OPTIONS as well?

[mmoroz, 2016-08-29 12:40:00]

Thanks Nico for comments!

https://codereview.chromium.org/2288753002/diff/1/testing/libfuzzer/reproduci...
File testing/libfuzzer/reproducing.md (right):

https://codereview.chromium.org/2288753002/diff/1/testing/libfuzzer/reproduci...
testing/libfuzzer/reproducing.md:93: make sure `llvm-symbolizer` is in `PATH` or
provide it in separate
On 2016/08/29 12:04:13, Nico wrote:
> Mention that llvm-symbolizer is in third_party/llvm-build/Release+Asserts/bin
> and/or that it's copied to the build dir at build time (and make the example
> point there)

Done.

https://codereview.chromium.org/2288753002/diff/1/testing/libfuzzer/reproduci...
testing/libfuzzer/reproducing.md:109: Please checkout
[AddressSanitizerAndDebugger] page for some tips on debugging of
On 2016/08/29 12:04:13, Nico wrote:
> s/checkout/look at/ since "checkout" sounds like `svn checkout`. (Also,
> "checkout" is the noun, while the verb is "check out" with a space).
> 
> Doesn't abort_on_error work in ASAN_OPTIONS as well?

Thanks for the explanation! My bad.

abort_on_error works for ASan too. My intention was to do not write about that,
since the link contains that information. But probably it's worth to be
mentioned here. Done.

[Nico, 2016-08-29 13:08:39]

lgtm, thanks!

[inferno, 2016-08-29 14:57:17]

The CQ bit was checked by inferno@chromium.org

[inferno, 2016-08-29 14:57:18]

lgtm

[commit-bot: I haz the power, 2016-08-29 14:57:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Oliver Chang, 2016-08-29 15:29:10]

lgtm

[commit-bot: I haz the power, 2016-08-29 15:49:20]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-29 15:51:08]

Description was changed from

==========
[libfuzzer] Add "Symbolization" and "Debugging" sections to reproducing.md page.

R=aizatsky@chromium.org, inferno@chromium.org, ochang@chromium.org,
thakis@chromium.org
BUG=640967
==========

to

==========
[libfuzzer] Add "Symbolization" and "Debugging" sections to reproducing.md page.

R=aizatsky@chromium.org, inferno@chromium.org, ochang@chromium.org,
thakis@chromium.org
BUG=640967

Committed: https://crrev.com/03e2e556db0bd18809f75059d0fc1804324d0e82
Cr-Commit-Position: refs/heads/master@{#415014}
==========

[commit-bot: I haz the power, 2016-08-29 15:51:09]

Patchset 3 (id:??) landed as
https://crrev.com/03e2e556db0bd18809f75059d0fc1804324d0e82
Cr-Commit-Position: refs/heads/master@{#415014}