This is an archive of the discontinued LLVM Phabricator instance.

Add build script for symbolizer which can be linked into instrumented process.
ClosedPublic

Authored by vitalybuka on Sep 27 2016, 11:41 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

vitalybuka retitled this revision from to Add build script for symbolizer which can be linked into instrumented process..
vitalybuka updated this object.
vitalybuka added a reviewer: eugenis.
vitalybuka added a subscriber: llvm-commits.
eugenis added inline comments.Sep 27 2016, 12:08 PM
lib/sanitizer_common/symbolizer/scripts/build_symbolizer.sh
61 ↗(On Diff #72681)

this comment does not make sense

68 ↗(On Diff #72681)

please remove -w

132 ↗(On Diff #72681)

this looks fragile
is it possible to unpack LLVM*.a instead?

137 ↗(On Diff #72681)

the build should use sanitizer_symbolizer_hooks.cc in some way

vitalybuka marked 3 inline comments as done.

Comments

vitalybuka updated this revision to Diff 72742.Sep 27 2016, 5:31 PM

fixed test

vitalybuka marked an inline comment as done.Sep 27 2016, 5:42 PM
eugenis added inline comments.Sep 27 2016, 5:54 PM
lib/sanitizer_common/symbolizer/sanitizer_symbolize.cc
21 ↗(On Diff #72743)

nullptr

lib/sanitizer_common/symbolizer/scripts/build_symbolizer.sh
15 ↗(On Diff #72743)

Please update the comment.

test/asan/TestCases/Linux/print_memory_profile_test.cc
33 ↗(On Diff #72743)

I don't understand the effect of this change.

vitalybuka marked an inline comment as done.

update

vitalybuka marked an inline comment as done.Sep 28 2016, 11:18 AM
eugenis accepted this revision.Sep 28 2016, 11:38 AM
eugenis edited edge metadata.

LGTM

lib/sanitizer_common/symbolizer/sanitizer_symbolize.cc
21 ↗(On Diff #72866)

I don't think you need this initializer at all.

This revision is now accepted and ready to land.Sep 28 2016, 11:38 AM
vitalybuka marked an inline comment as done.
vitalybuka edited edge metadata.

initialization

This revision was automatically updated to reflect the committed changes.