This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Symbolizer refactoring: Move internals to separate files
ClosedPublic

Authored by kubamracek on Feb 28 2015, 3:11 AM.

Details

Reviewers
samsonov
Summary

Part of http://reviews.llvm.org/D7827. Let's introduce a sanitizer_symbolizer_internal.h header file that will contain definitions and classes that are used by symbolizers, and keep sanitizer_symbolizer.h as the public interface for the sanitizers. This also moves the implementation of SymbolizerProcess into a separate file, sanitizer_symbolizer_process_libcdep.cc.

Diff Detail

Event Timeline

kubamracek updated this revision to Diff 20933.Feb 28 2015, 3:11 AM
kubamracek retitled this revision from to [compiler-rt] Symbolizer refactoring: Move internals to separate files.
kubamracek updated this object.
kubamracek edited the test plan for this revision. (Show Details)
kubamracek added subscribers: Unknown Object (MLST), zaks.anna, samsonov.
samsonov accepted this revision.Mar 2 2015, 12:07 PM
samsonov added a reviewer: samsonov.

LGTM

This revision is now accepted and ready to land.Mar 2 2015, 12:07 PM
kubamracek closed this revision.Mar 2 2015, 1:23 PM

Landed in r231014.