[llvm] [DWARFDump][nfc] Fix incorrect comment (PR #81276)

Felipe de Azevedo Piovezan via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 9 08:53:22 PST 2024


https://github.com/felipepiovezan updated https://github.com/llvm/llvm-project/pull/81276

>From 0583356b2f669a1ca3352704be2a075d5fa7a121 Mon Sep 17 00:00:00 2001
From: Felipe de Azevedo Piovezan <fpiovezan at apple.com>
Date: Fri, 9 Feb 2024 08:39:25 -0800
Subject: [PATCH] [DWARFDump][nfc] Fix incorrect comment

It claimed to dump all sections by default, but this hasn't been true since
2017: https://reviews.llvm.org/D37717
---
 llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp b/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
index 559e7a60489834..8cdd84bcc867cb 100644
--- a/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
+++ b/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
@@ -845,8 +845,9 @@ int main(int argc, char **argv) {
 
   bool OffsetRequested = false;
 
-  // Defaults to dumping all sections, unless brief mode is specified in which
-  // case only the .debug_info section in dumped.
+  // Defaults to dumping only debug_info, unless: A) verbose mode is specified,
+  // in which case all sections are dumped, or B) a specific section is
+  // requested.
 #define HANDLE_DWARF_SECTION(ENUM_NAME, ELF_NAME, CMDLINE_NAME, OPTION)        \
   if (Dump##ENUM_NAME.IsRequested) {                                           \
     DumpType |= DIDT_##ENUM_NAME;                                              \



More information about the llvm-commits mailing list