[Lldb-commits] [PATCH] D54617: [Reproducers] Add file provider

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jan 17 10:30:05 PST 2019


JDevlieghere marked an inline comment as done.
JDevlieghere added inline comments.


================
Comment at: include/lldb/Utility/Reproducer.h:91-92
+
+const char *FileInfo::name = "files";
+const char *FileInfo::file = "files.yaml";
+
----------------
labath wrote:
> Are you sure this can be in a header? I would expect this to give you multiply-defined symbols errors.
You are correct, should be in the implementation. 


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

https://reviews.llvm.org/D54617





More information about the lldb-commits mailing list