Skip to content

Commit

Permalink
[tsan] Track external API accesses as 1-byte accesses (instead of 8-b…
Browse files Browse the repository at this point in the history
…yte)

It doesn't really make sense to track them as 8-byte accesses.

Differential Revision: https://reviews.llvm.org/D32359

llvm-svn: 301001
  • Loading branch information
kubamracek committed Apr 21, 2017
1 parent 2e4e7d0 commit 676d008
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions compiler-rt/lib/tsan/rtl/tsan_external.cc
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ void __tsan_external_read(void *addr, void *caller_pc, void *tag) {
if (caller_pc) FuncEntry(thr, (uptr)caller_pc);
bool in_ignored_lib;
if (!caller_pc || !libignore()->IsIgnored((uptr)caller_pc, &in_ignored_lib)) {
MemoryRead(thr, CALLERPC, (uptr)addr, kSizeLog8);
MemoryRead(thr, CALLERPC, (uptr)addr, kSizeLog1);
}
if (caller_pc) FuncExit(thr);
thr->external_tag = 0;
Expand All @@ -75,7 +75,7 @@ void __tsan_external_write(void *addr, void *caller_pc, void *tag) {
if (caller_pc) FuncEntry(thr, (uptr)caller_pc);
bool in_ignored_lib;
if (!caller_pc || !libignore()->IsIgnored((uptr)caller_pc, &in_ignored_lib)) {
MemoryWrite(thr, CALLERPC, (uptr)addr, kSizeLog8);
MemoryWrite(thr, CALLERPC, (uptr)addr, kSizeLog1);
}
if (caller_pc) FuncExit(thr);
thr->external_tag = 0;
Expand Down

0 comments on commit 676d008

Please sign in to comment.