[PATCH] D58621: [XRay][tools] Pack XRayRecord - reduce memory footprint by a third. (RFC)

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 8 15:53:35 PST 2019


dberris added a comment.

In D58621#1774144 <https://reviews.llvm.org/D58621#1774144>, @nbaksalyar wrote:

> In D58621#1409962 <https://reviews.llvm.org/D58621#1409962>, @dberris wrote:
>
> > Now, an alternative here is to migrate the Basic Mode implementation to use a more compact log record (i.e. using the FDR mode format), and use a different converter approach <..> This is a more intensive project but one that isn't terribly hard to accomplish. If you'd like to take that on, I'd be happy to review patches going in that direction instead
>
>
> @dberris, is this still valid & desirable?
>  I would like to dip my toes into XRay/LLVM, and this seems like a good first mini project to me. :) This migration should also bring typed and custom events support for the basic mode, which is a nice bonus.


Yes, still valid and still desirable. :)


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58621/new/

https://reviews.llvm.org/D58621





More information about the llvm-commits mailing list