[PATCH] D53802: [XRay] Use more portable control block

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 28 19:11:26 PDT 2018


dberris created this revision.
dberris added reviewers: mboerger, eizan.
Herald added a subscriber: jfb.

In https://reviews.llvm.org/D53560, we assumed a specific layout for memory without using an
explicit structure. This follow-up change uses more portable layout
control by using unions in a struct, and consolidating the memory
management code in the buffer queue.

We also take the opportunity to improve the documentation on the types
and operations, along with simplifying some of the logic in the buffer
queue implementation.


https://reviews.llvm.org/D53802

Files:
  compiler-rt/lib/xray/tests/unit/buffer_queue_test.cc
  compiler-rt/lib/xray/xray_buffer_queue.cc
  compiler-rt/lib/xray/xray_buffer_queue.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53802.171445.patch
Type: text/x-patch
Size: 9102 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181029/a0c9d44b/attachment.bin>


More information about the llvm-commits mailing list