[PATCH] D58621: [XRay][tools] Pack XRayRecord - reduce memory footprint by a third. (RFC)
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 25 13:25:14 PST 2019
lebedev.ri planned changes to this revision.
lebedev.ri added a comment.
In D58621#1409704 <https://reviews.llvm.org/D58621#1409704>, @dberris wrote:
> > This is a RFC because of the uint8_t CPU change.
> > That chance needs discussing.
>
> So, this is an accident of history, which should be changed, but to the other direction. I've learned some time ago that it turns out there are some platforms that can have enough CPU IDs which can't be represented by a uint8_t. For future-proofing, we really should change this to be larger (`uint16_t`) and change basic mode to store 16-bit CPU IDs.
Boo :)
Unfortunately that won't just cost that one extra byte, it will have ripple effect on the padding in this struct.
I'm not sure as to exact numbers.
> The other parts seem fine to me, except for the potential churn on the user side (this is an ABI change).
>
> I //think// that's fine for the C++ APIs, but that it will need some release notes (in case someone has been using the `Trace` API).
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