[PATCH] D32840: [XRay] convert FDR arg1 log entries
Dean Michael Berris via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 17 21:18:28 PDT 2017
dberris added a comment.
I'm adding support for tail exit records, can you wait on submitting this as this changes a number of similar places?
================
Comment at: include/llvm/XRay/YAMLXRayRecord.h:89
+template <> struct SequenceTraits<std::vector<uint64_t>> {
+ static constexpr bool flow = true;
+ static size_t size(IO &IO, std::vector<uint64_t> &V) { return V.size(); }
----------------
What is this meant to do? Is it meant to force text wrapping? If so, please don't do that (it messes with the tests and the settings that can be defined on the YAML stream instead).
https://reviews.llvm.org/D32840
More information about the llvm-commits
mailing list