labath added a comment.

This looks fine, but I think you'll first need to handle the lit changes to 
allow skipping tests. Otherwise all of the pdb and breakpad tests will break 
once you do this. I guess the reason that this hasn't happened yet is because 
you have a typo in your cache file.



================
Comment at: lldb/source/API/SBCommandReturnObject.cpp:98-101
+  std::ofstream reproducer_log_file;
+  
reproducer_log_file.open(lldb_private::repro::Reproducer::Instance().IsCapturing()
 ? "/tmp/capture.log" : "/tmp/replay.log", std::fstream::app);
+  reproducer_log_file << output.AsCString(/*value_if_empty*/ "") << "\n";
+  reproducer_log_file.close();
----------------
?


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

https://reviews.llvm.org/D73016



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to