[PATCH] D50867: [InstrProf] Use atomic profile counter updates for TSan

[Vedant Kumar via Phabricator via llvm-commits]

vsk added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp:608
+    Builder.CreateAtomicRMW(AtomicRMWInst::Add, Addr, Inc->getStep(),
+                            AtomicOrdering::SequentiallyConsistent);
+  } else {
----------------
jfb wrote:
> Counters could just be `relaxed`, no need to use `seq_cst`.
Thanks for calling this out.


https://reviews.llvm.org/D50867