[PATCH] D29703: [XRay] [compiler-rt] Allow logging the first argument of a function call.

[Martin Pelikán via Phabricator via llvm-commits]

pelikan added inline comments.


================
Comment at: lib/xray/xray_interface.cc:217
+{
+  XRayArgLogger.store(Handler, std::memory_order_relaxed);
+}
----------------
dberris wrote:
> I think it would be better to make this store a release, for a stronger guarantee. 
Stronger guarantee of what?  There's nothing we're synchronizing *with*, we only care about atomicity (which this provides).  When I see a release/consume I automatically look for an acquire, and in this case there isn't one.


https://reviews.llvm.org/D29703