[compiler-rt] [TSan] Make Shadow/Meta region inclusive-exclusive (PR #144647)
Kunqiu Chen via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 18 02:04:54 PDT 2025
================
@@ -675,7 +675,7 @@ void MemoryAccessRangeT(ThreadState* thr, uptr pc, uptr addr, uptr size) {
Printf("Access to non app mem start: %p\n", (void*)addr);
DCHECK(IsAppMem(addr));
}
- if (!IsAppMem(addr + size - 1)) {
+ if (size > 0 && !IsAppMem(addr + size - 1)) {
----------------
Camsyn wrote:
> But it's clear currently an invariant here is `size != 0`, so why not add a DCHECK and catch if that assumption changes in future?
Done.
https://github.com/llvm/llvm-project/pull/144647
More information about the llvm-commits
mailing list