This is an archive of the discontinued LLVM Phabricator instance.

Move parts of llvm-symbolizer tool into LLVMSymbolize library.
ClosedPublic

Authored by samsonov on Oct 22 2015, 3:23 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

samsonov updated this revision to Diff 38183.Oct 22 2015, 3:23 PM
samsonov retitled this revision from to Move parts of llvm-symbolizer tool into LLVMSymbolize library..
samsonov updated this object.
samsonov added a reviewer: echristo.
samsonov added subscribers: aizatsky, llvm-commits.

Couple of inline comments otherwise it looks fine to me.

Rafael: You had objected originally about this, do you still mind?

-eric

lib/DebugInfo/Symbolize/Symbolize.cpp
14 ↗(On Diff #38183)

Should be down with the other headers?

49 ↗(On Diff #38183)

Please do :)

echristo accepted this revision.Oct 23 2015, 5:40 PM
echristo edited edge metadata.
This revision is now accepted and ready to land.Oct 23 2015, 5:40 PM

Thanks!

lib/DebugInfo/Symbolize/Symbolize.cpp
14 ↗(On Diff #38183)
49 ↗(On Diff #38183)

I'll do it in a follow-up changes: for now LLVMSymbolizer doesn't know how to return errors to the user, and it would take some refactoring to achieve that - let's keep this change a relatively straightforward code move.

This revision was automatically updated to reflect the committed changes.

Thanks, Alexey. I'm definitely going to use it in the improved version of sanitizer coverage tool.

llvm/trunk/lib/DebugInfo/Symbolize/Symbolize.cpp