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

[Dmitry Vyukov via llvm-commits]

dvyukov added a comment.

> It is not about this assumption, but more about racy counter updates not being a problem practically


Choosing an incorrect code all others equal does not look like the right approach to me.

> It might -- at least we should measure it. Note that instrumentation run is already pretty slow, so we need to be careful not to regress it.


Again, is that's the case, that's a very real bug in llvm codegen that needs to be fixed. Patching all uses of atomic operations in the world to work around that bug is not going to scale (and just wrong).


http://reviews.llvm.org/D18164