[lldb] Remove LLDB_CAPTURE_REPRODUCER override
Remove the LLDB_CAPTURE_REPRODUCER as it is inherently dangerous. The
reproducers require careful initialization which cannot be guaranteed by
overwriting the reproducer mode at this level.
If we want to provide this functionality, we should do it in the driver
instead. It was originally added to enable capture in CI, but we now
have a dedicated CI job that captures and replays the test suite.
GitOrigin-RevId: dfc8da19c5d7ed48f62a4cc588b641f2deee8789
diff --git a/source/Utility/Reproducer.cpp b/source/Utility/Reproducer.cpp
index d9207b1..b63863c 100644
--- a/source/Utility/Reproducer.cpp
+++ b/source/Utility/Reproducer.cpp
@@ -20,15 +20,6 @@
using namespace llvm;
using namespace llvm::yaml;
-static llvm::Optional<bool> GetEnv(const char *var) {
- std::string val = llvm::StringRef(getenv(var)).lower();
- if (val == "0" || val == "off")
- return false;
- if (val == "1" || val == "on")
- return true;
- return {};
-}
-
Reproducer &Reproducer::Instance() { return *InstanceImpl(); }
llvm::Error Reproducer::Initialize(ReproducerMode mode,
@@ -36,16 +27,6 @@
lldbassert(!InstanceImpl() && "Already initialized.");
InstanceImpl().emplace();
- // The environment can override the capture mode.
- if (mode != ReproducerMode::Replay) {
- if (llvm::Optional<bool> override = GetEnv("LLDB_CAPTURE_REPRODUCER")) {
- if (*override)
- mode = ReproducerMode::Capture;
- else
- mode = ReproducerMode::Off;
- }
- }
-
switch (mode) {
case ReproducerMode::Capture: {
if (!root) {
