[lldb] Remove LLDB reproducers

This patch removes the remaining reproducer code. The SBReproducer class
remains for ABI stability but is just an empty shell. This completes the
removal process outlined on the mailing list [1].

[1] https://lists.llvm.org/pipermail/lldb-dev/2021-September/017045.html

Author: JDevlieghere

lldb/include/lldb/API/SBReproducer.h

@@ -32,9 +32,6 @@ class LLDB_API SBReplayOptions {
 
   void SetCheckVersion(bool check);
   bool GetCheckVersion() const;
-
-private:
-  std::unique_ptr<lldb_private::repro::ReplayOptions> m_opaque_up;
 };
 
 /// The SBReproducer class is special because it bootstraps the capture and

lldb/include/lldb/Core/Debugger.h

@@ -152,11 +152,7 @@ class Debugger : public std::enable_shared_from_this<Debugger>,
 
   Status SetInputString(const char *data);
 
-  // This method will setup data recorder if reproducer enabled.
-  // On reply mode this method should take instructions from reproducer file.
-  Status SetInputFile(lldb::FileSP file);
-
-  void SetInputFile(lldb::FileSP file, repro::DataRecorder *recorder);
+  void SetInputFile(lldb::FileSP file);
 
   void SetOutputFile(lldb::FileSP file);
 
@@ -294,8 +290,6 @@ class Debugger : public std::enable_shared_from_this<Debugger>,
   void SetPrompt(llvm::StringRef p);
   void SetPrompt(const char *) = delete;
 
-  llvm::StringRef GetReproducerPath() const;
-
   bool GetUseExternalEditor() const;
 
   bool SetUseExternalEditor(bool use_external_editor_p);

lldb/include/lldb/Core/IOHandler.h

@@ -31,9 +31,6 @@
 
 namespace lldb_private {
 class Debugger;
-namespace repro {
-class DataRecorder;
-}
 } // namespace lldb_private
 
 namespace curses {
@@ -63,8 +60,7 @@ class IOHandler {
 
   IOHandler(Debugger &debugger, IOHandler::Type type,
             const lldb::FileSP &input_sp, const lldb::StreamFileSP &output_sp,
-            const lldb::StreamFileSP &error_sp, uint32_t flags,
-            repro::DataRecorder *data_recorder);
+            const lldb::StreamFileSP &error_sp, uint32_t flags);
 
   virtual ~IOHandler();
 
@@ -173,7 +169,6 @@ class IOHandler {
   lldb::StreamFileSP m_output_sp;
   lldb::StreamFileSP m_error_sp;
   std::recursive_mutex m_output_mutex;
-  repro::DataRecorder *m_data_recorder;
   Predicate<bool> m_popped;
   Flags m_flags;
   Type m_type;
@@ -338,8 +333,7 @@ class IOHandlerEditline : public IOHandler {
                     uint32_t line_number_start, // If non-zero show line numbers
                                                 // starting at
                                                 // 'line_number_start'
-                    IOHandlerDelegate &delegate,
-                    repro::DataRecorder *data_recorder);
+                    IOHandlerDelegate &delegate);
 
   IOHandlerEditline(Debugger &debugger, IOHandler::Type type,
                     const lldb::FileSP &input_sp,
@@ -351,8 +345,7 @@ class IOHandlerEditline : public IOHandler {
                     uint32_t line_number_start, // If non-zero show line numbers
                                                 // starting at
                                                 // 'line_number_start'
-                    IOHandlerDelegate &delegate,
-                    repro::DataRecorder *data_recorder);
+                    IOHandlerDelegate &delegate);
 
   IOHandlerEditline(Debugger &, IOHandler::Type, const char *, const char *,
                     const char *, bool, bool, uint32_t,

lldb/include/lldb/Interpreter/CommandOptionArgumentTable.h

@@ -124,67 +124,6 @@ static constexpr OptionEnumValueElement g_log_handler_type[] = {
     },
 };
 
-static constexpr OptionEnumValueElement g_reproducer_provider_type[] = {
-    {
-        eReproducerProviderCommands,
-        "commands",
-        "Command Interpreter Commands",
-    },
-    {
-        eReproducerProviderFiles,
-        "files",
-        "Files",
-    },
-    {
-        eReproducerProviderSymbolFiles,
-        "symbol-files",
-        "Symbol Files",
-    },
-    {
-        eReproducerProviderGDB,
-        "gdb",
-        "GDB Remote Packets",
-    },
-    {
-        eReproducerProviderProcessInfo,
-        "processes",
-        "Process Info",
-    },
-    {
-        eReproducerProviderVersion,
-        "version",
-        "Version",
-    },
-    {
-        eReproducerProviderWorkingDirectory,
-        "cwd",
-        "Working Directory",
-    },
-    {
-        eReproducerProviderHomeDirectory,
-        "home",
-        "Home Directory",
-    },
-    {
-        eReproducerProviderNone,
-        "none",
-        "None",
-    },
-};
-
-static constexpr OptionEnumValueElement g_reproducer_signaltype[] = {
-    {
-        eReproducerCrashSigill,
-        "SIGILL",
-        "Illegal instruction",
-    },
-    {
-        eReproducerCrashSigsegv,
-        "SIGSEGV",
-        "Segmentation fault",
-    },
-};
-
 static constexpr OptionEnumValueElement g_script_synchro_type[] = {
     {
         eScriptedCommandSynchronicitySynchronous,
@@ -318,8 +257,6 @@ static constexpr CommandObject::ArgumentTableEntry g_argument_table[] = {
     { lldb::eArgTypeConnectURL, "process-connect-url", CommandCompletions::eNoCompletion, {}, { nullptr, false }, "A URL-style specification for a remote connection." },
     { lldb::eArgTypeTargetID, "target-id", CommandCompletions::eNoCompletion, {}, { nullptr, false }, "The index ID for an lldb Target." },
     { lldb::eArgTypeStopHookID, "stop-hook-id", CommandCompletions::eNoCompletion, {}, { nullptr, false }, "The ID you receive when you create a stop-hook." },
-    { lldb::eArgTypeReproducerProvider, "reproducer-provider", CommandCompletions::eNoCompletion, g_reproducer_provider_type, { nullptr, false }, "The reproducer provider." },
-    { lldb::eArgTypeReproducerSignal, "reproducer-signal", CommandCompletions::eNoCompletion, g_reproducer_signaltype, { nullptr, false }, "The signal used to emulate a reproducer crash." },
     // clang-format on
 };
 

lldb/include/lldb/Utility/ArchSpec.h

@@ -16,7 +16,6 @@
 #include "lldb/lldb-private-enumerations.h"
 #include "llvm/ADT/StringRef.h"
 #include "llvm/ADT/Triple.h"
-#include "llvm/Support/YAMLTraits.h"
 #include <cstddef>
 #include <cstdint>
 #include <string>
@@ -568,16 +567,4 @@ bool ParseMachCPUDashSubtypeTriple(llvm::StringRef triple_str, ArchSpec &arch);
 
 } // namespace lldb_private
 
-namespace llvm {
-namespace yaml {
-template <> struct ScalarTraits<lldb_private::ArchSpec> {
-  static void output(const lldb_private::ArchSpec &, void *, raw_ostream &);
-  static StringRef input(StringRef, void *, lldb_private::ArchSpec &);
-  static QuotingType mustQuote(StringRef S) { return QuotingType::Double; }
-};
-} // namespace yaml
-} // namespace llvm
-
-LLVM_YAML_IS_SEQUENCE_VECTOR(lldb_private::ArchSpec)
-
 #endif // LLDB_UTILITY_ARCHSPEC_H

lldb/include/lldb/Utility/Args.h

@@ -35,8 +35,6 @@ class Args {
   struct ArgEntry {
   private:
     friend class Args;
-    friend struct llvm::yaml::MappingTraits<Args>;
-    friend struct llvm::yaml::MappingTraits<Args::ArgEntry>;
 
     std::unique_ptr<char[]> ptr;
     char quote = '\0';
@@ -287,8 +285,6 @@ class Args {
                                                char quote_char);
 
 private:
-  friend struct llvm::yaml::MappingTraits<Args>;
-
   std::vector<ArgEntry> m_entries;
   /// The arguments as C strings with a trailing nullptr element.
   ///
@@ -383,28 +379,4 @@ class OptionsWithRaw {
 
 } // namespace lldb_private
 
-namespace llvm {
-namespace yaml {
-template <> struct MappingTraits<lldb_private::Args::ArgEntry> {
-  class NormalizedArgEntry {
-  public:
-    NormalizedArgEntry(IO &) {}
-    NormalizedArgEntry(IO &, lldb_private::Args::ArgEntry &entry)
-        : value(entry.ref()), quote(entry.quote) {}
-    lldb_private::Args::ArgEntry denormalize(IO &) {
-      return lldb_private::Args::ArgEntry(value, quote);
-    }
-    StringRef value;
-    uint8_t quote = '\0';
-  };
-  static void mapping(IO &io, lldb_private::Args::ArgEntry &v);
-};
-template <> struct MappingTraits<lldb_private::Args> {
-  static void mapping(IO &io, lldb_private::Args &v);
-};
-} // namespace yaml
-} // namespace llvm
-
-LLVM_YAML_IS_SEQUENCE_VECTOR(lldb_private::Args::ArgEntry)
-
 #endif // LLDB_UTILITY_ARGS_H

lldb/include/lldb/Utility/ConstString.h

@@ -12,7 +12,6 @@
 #include "llvm/ADT/DenseMapInfo.h"
 #include "llvm/ADT/StringRef.h"
 #include "llvm/Support/FormatVariadic.h"
-#include "llvm/Support/YAMLTraits.h"
 
 #include <cstddef>
 
@@ -450,20 +449,10 @@ template <> struct DenseMapInfo<lldb_private::ConstString> {
 };
 /// \}
 
-namespace yaml {
-template <> struct ScalarTraits<lldb_private::ConstString> {
-  static void output(const lldb_private::ConstString &, void *, raw_ostream &);
-  static StringRef input(StringRef, void *, lldb_private::ConstString &);
-  static QuotingType mustQuote(StringRef S) { return QuotingType::Double; }
-};
-} // namespace yaml
-
 inline raw_ostream &operator<<(raw_ostream &os, lldb_private::ConstString s) {
   os << s.GetStringRef();
   return os;
 }
 } // namespace llvm
 
-LLVM_YAML_IS_SEQUENCE_VECTOR(lldb_private::ConstString)
-
 #endif // LLDB_UTILITY_CONSTSTRING_H

lldb/include/lldb/Utility/FileSpec.h

@@ -18,7 +18,6 @@
 #include "llvm/Support/FileSystem.h"
 #include "llvm/Support/FormatVariadic.h"
 #include "llvm/Support/Path.h"
-#include "llvm/Support/YAMLTraits.h"
 
 #include <cstddef>
 #include <cstdint>
@@ -411,8 +410,6 @@ class FileSpec {
   ConstString GetLastPathComponent() const;
 
 protected:
-  friend struct llvm::yaml::MappingTraits<FileSpec>;
-
   // Convenience method for setting the file without changing the style.
   void SetFile(llvm::StringRef path);
 
@@ -439,9 +436,6 @@ class FileSpec {
 
 /// Dump a FileSpec object to a stream
 Stream &operator<<(Stream &s, const FileSpec &f);
-
-/// Prevent ODR violations with traits for llvm::sys::path::Style.
-LLVM_YAML_STRONG_TYPEDEF(FileSpec::Style, FileSpecStyle)
 } // namespace lldb_private
 
 namespace llvm {
@@ -469,15 +463,6 @@ template <> struct format_provider<lldb_private::FileSpec> {
                      StringRef Style);
 };
 
-namespace yaml {
-template <> struct ScalarEnumerationTraits<lldb_private::FileSpecStyle> {
-  static void enumeration(IO &io, lldb_private::FileSpecStyle &style);
-};
-
-template <> struct MappingTraits<lldb_private::FileSpec> {
-  static void mapping(IO &io, lldb_private::FileSpec &f);
-};
-} // namespace yaml
 } // namespace llvm
 
 #endif // LLDB_UTILITY_FILESPEC_H