This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Add check to make sure that located PC is reasonable close.
ClosedPublic

Authored by vitalybuka on Jan 5 2017, 1:18 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

vitalybuka updated this revision to Diff 83293.Jan 5 2017, 1:18 PM
vitalybuka retitled this revision from to [compiler-rt] Add check to make sure that located PC is reasonable close..
vitalybuka updated this object.
vitalybuka added a reviewer: eugenis.
vitalybuka added a subscriber: llvm-commits.
vitalybuka updated this revision to Diff 83295.Jan 5 2017, 1:22 PM

Check for empty size

eugenis accepted this revision.Jan 5 2017, 1:23 PM
eugenis edited edge metadata.
eugenis added inline comments.
lib/asan/tests/asan_interface_test.cc
15 ↗(On Diff #83293)

sanitizer_common header should be in quotes (").
Also, it should go on top, along with asan_test_utils.h:
http://llvm.org/docs/CodingStandards.html#include-style

This revision is now accepted and ready to land.Jan 5 2017, 1:23 PM
vitalybuka updated this revision to Diff 83297.Jan 5 2017, 1:34 PM
vitalybuka edited edge metadata.

Just update the test

This revision was automatically updated to reflect the committed changes.
vitalybuka marked an inline comment as done.Jan 5 2017, 1:37 PM