First, the define conflicts with definition/testing of all mappings,
since it's not a global property anymore. Second, macros/ifdefs are bad.
Define kMidAppMemBeg/End to 0 to denote that there is no "mid" range instead.
Depends on D107736.
Paths
| Differential D107737
tsan: remove TSAN_MID_APP_RANGE macro ClosedPublic Authored by dvyukov on Aug 9 2021, 12:50 AM.
Details
Summary First, the define conflicts with definition/testing of all mappings, Depends on D107736.
Diff Detail
Event Timelinedvyukov added a child revision: D107738: tsan: rename kAppMemBeg to kLoAppMemBeg.Aug 9 2021, 12:50 AM This revision is now accepted and ready to land.Aug 9 2021, 3:53 AM dvyukov retitled this revision from tsan: remote TSAN_MID_APP_RANGE macro to tsan: remove TSAN_MID_APP_RANGE macro.Aug 9 2021, 10:22 AM Closed by commit rGfda121a9cc7e: tsan: remove TSAN_MID_APP_RANGE macro (authored by dvyukov). · Explain WhyAug 10 2021, 11:04 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 365552 compiler-rt/lib/tsan/rtl/tsan_platform.h
compiler-rt/lib/tsan/rtl/tsan_platform_posix.cpp
|