[Lldb-commits] [PATCH] D85705: Add a "Trace" plug-in to LLDB to add process trace support in stages.

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere added a comment.

A large part of this patch is concerned with parsing which worries me from a maintenance perspective. Did you consider using Yaml I/O <https://llvm.org/docs/YamlIO.html>? While I'm not a particularly big fan of the format, the benefits of being able to (de)serialize any class by implementing the appropriate traits are quite significant. We already have traits implemented for a bunch of utility classes, such as `ArchSpec` and `FileSpec` which we could reuse for this. I know changing the format would be invasive, but I think it might be worth it in the long term.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85705