Page MenuHomePhabricator

[asan] Move `DescribeSignalOrException` to sanitizer common.
ClosedPublic

Authored by mpividori on Feb 2 2017, 9:28 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

mpividori created this revision.Feb 2 2017, 9:28 AM
kubamracek added inline comments.Feb 2 2017, 9:32 AM
lib/asan/asan_errors.cc
64 ↗(On Diff #86832)

Why do we need __sanitizer::? Other parts of ASan call sanitizer_common code all the time without this.

rnk accepted this revision.Feb 2 2017, 9:48 AM

lgtm

This revision is now accepted and ready to land.Feb 2 2017, 9:48 AM
mpividori added inline comments.Feb 2 2017, 10:57 AM
lib/asan/asan_errors.cc
64 ↗(On Diff #86832)

You are right. Is not necessary.

This revision was automatically updated to reflect the committed changes.