[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