This is an archive of the discontinued LLVM Phabricator instance.

[ASan] Make binary name reader cross-platform.
ClosedPublic

Authored by ygribov on Jun 3 2015, 7:13 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ygribov updated this revision to Diff 27037.Jun 3 2015, 7:13 AM
ygribov retitled this revision from to [ASan] Make binary name reader cross-platform..
ygribov updated this object.
ygribov edited the test plan for this revision. (Show Details)
ygribov added reviewers: samsonov, kcc, dvyukov.
ygribov set the repository for this revision to rL LLVM.
ygribov added a subscriber: Unknown Object (MLST).
samsonov accepted this revision.Jun 3 2015, 9:19 AM
samsonov edited edge metadata.

LGTM with a couple of comments

lib/sanitizer_common/sanitizer_common.cc
353 ↗(On Diff #27037)
CHECK_EQ('\0', binary_name_cache_str[0]);
lib/sanitizer_common/sanitizer_common.h
244 ↗(On Diff #27037)

This function must have been declared in some other header. Delete it from there.

This revision is now accepted and ready to land.Jun 3 2015, 9:19 AM
This revision was automatically updated to reflect the committed changes.