[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