[Lldb-commits] [PATCH] D77602: [lldb/Reproducers] Support inline replay
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Apr 9 22:17:57 PDT 2020
JDevlieghere updated this revision to Diff 256505.
JDevlieghere added a comment.
- Simplify `LLDB_RECORD_CONSTRUCTOR` macro. The other macros have `return` statements that need to be inlined in the caller for replay, and the boundary tracking needs to be updated right, so I'm not convinced the extra complexity is worth the deduplication.
- Use the replayer ID to check the signature.
I didn't get around to adding the unit tests yet. I noticed that this change breaks something and I'm still debugging it. I think it's specific to the unit test though, as the test suite isn't affected.
CHANGES SINCE LAST ACTION
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 39448 bytes
Desc: not available
More information about the lldb-commits