[Lldb-commits] [PATCH] D56814: [Reproducers] Refactor reproducer info
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Jan 16 15:02:26 PST 2019
JDevlieghere created this revision.
JDevlieghere added reviewers: labath, davide.
Herald added subscribers: abidh, mgrang.
In the original reproducer design, I expected providers to be more dynamic than they turned out. For example, we don't have any instances where one provider has multiple files. Additionally, I expected there to be less locality between capture and replay, with the provider being defined in one place and the replay code to live in another. Both contributed to the design of the provider info.
This patch refactors the reproducer info to be something static. This means less magic strings and better type checking. The new design still allows for the capture and replay code to live in different places as long as they both have access to the new statically defined info class.
I didn't completely get rid of the index, because it is useful for (1) sanity checking and (2) knowing what files are used by the reproducer.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6468 bytes
Desc: not available
More information about the lldb-commits