[PATCH] D54291: [XRay] Add atomic fences around non-atomic reads and writes

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 9 12:02:32 PST 2018


efriedma added a comment.

What is the point of this change?  The atomic_fetch_add operations already use memory_order_acq_rel, so the explicit fences are redundant.


Repository:
  rL LLVM

https://reviews.llvm.org/D54291





More information about the llvm-commits mailing list