From 49562d3dfed0ed6983d29d040db7e46ef3bc833e Mon Sep 17 00:00:00 2001 From: Arthur Eubanks Date: Sat, 16 Oct 2021 12:05:41 -0700 Subject: [PATCH] Revert "[clang] Pass -clear-ast-before-backend in Clang::ConstructJob()" This reverts commit 47eb99aa44ab1d20327d67a49d6c47163de76387. This causes crashes with -print-stats: PR52193. --- clang/lib/Driver/ToolChains/Clang.cpp | 1 - clang/lib/Interpreter/Interpreter.cpp | 4 ---- 2 files changed, 5 deletions(-) diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index 83afbc3..d2c0841 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -4660,7 +4660,6 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, // cleanup. if (!C.isForDiagnostics()) CmdArgs.push_back("-disable-free"); - CmdArgs.push_back("-clear-ast-before-backend"); #ifdef NDEBUG const bool IsAssertBuild = false; diff --git a/clang/lib/Interpreter/Interpreter.cpp b/clang/lib/Interpreter/Interpreter.cpp index d14940d..02b3025 100644 --- a/clang/lib/Interpreter/Interpreter.cpp +++ b/clang/lib/Interpreter/Interpreter.cpp @@ -113,10 +113,6 @@ CreateCI(const llvm::opt::ArgStringList &Argv) { Clang->getTarget().adjust(Clang->getDiagnostics(), Clang->getLangOpts()); - // Don't clear the AST before backend codegen since we do codegen multiple - // times, reusing the same AST. - Clang->getCodeGenOpts().ClearASTBeforeBackend = false; - return std::move(Clang); } -- 2.7.4