From 0e4b8b8f81c97ef7374c321fd67a804dd17b4532 Mon Sep 17 00:00:00 2001 From: Jeremy Morse Date: Mon, 9 Jun 2025 19:36:34 +0100 Subject: [DebugInfo][RemoveDIs] Rip out the UseNewDbgInfoFormat flag (#143207) Start removing debug intrinsics support -- starting with the flag that controls production of their replacement, debug records. This patch removes the command-line-flag and with it the ability to switch back to intrinsics. The module / function / block level "IsNewDbgInfoFormat" flags get hardcoded to true, I'll to incrementally remove things that depend on those flags. --- llvm/lib/Bitcode/Reader/BitcodeReader.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp') diff --git a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp index 105edb9..31129b7 100644 --- a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp +++ b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp @@ -101,8 +101,6 @@ static cl::opt ExpandConstantExprs( cl::desc( "Expand constant expressions to instructions for testing purposes")); -LLVM_ABI extern cl::opt UseNewDbgInfoFormat; - namespace { enum { @@ -4481,9 +4479,9 @@ Error BitcodeReader::parseGlobalIndirectSymbolRecord( Error BitcodeReader::parseModule(uint64_t ResumeBit, bool ShouldLazyLoadMetadata, ParserCallbacks Callbacks) { - // In preparation for the deletion of debug-intrinsics, don't allow module - // loading to escape intrinsics being autoupgraded to debug records. - TheModule->IsNewDbgInfoFormat = UseNewDbgInfoFormat; + // Don't allow modules to use debug-intrinsics: autoupgrading them is now + // mandatory. + TheModule->IsNewDbgInfoFormat = true; this->ValueTypeCallback = std::move(Callbacks.ValueType); if (ResumeBit) { -- cgit v1.1