[PATCH] D24376: [XRay] Implement `llvm-xray convert` -- trace file conversion
David Blaikie via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 27 14:30:13 PDT 2016
dblaikie added a comment.
A few of the inline comments from my last feedback are probably out of date/not relevant - I had them queued up from weeks/months ago.
That said - taking a guess at it: Yes, I'd use DataExtractor here over splatting/memcpying structs around for the same reasons.
https://reviews.llvm.org/D24376
More information about the llvm-commits
mailing list