[PATCH] D18164: [tsan] Do not instrument reads/writes to instruction profile counters.

[Anna Zaks via llvm-commits]

zaks.anna added a comment.

> so the use case include configs like:

> 

> - coverage + tsan

> - coverage + asan

> - coverage + ...san ?


Correct.

> In the small case I tried, tsan  replaces the monotonic load with __tsan_atomic_load32(..) call -- i still

>  saw  the runtime warning.


Could you send me the test case? In the small test case I tried, I did not see a runtime warning:) Also, have you made both the load and the store monotonic?


http://reviews.llvm.org/D18164