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

via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 9 08:52:02 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-debuginfo

Author: Felipe de Azevedo Piovezan (felipepiovezan)

<details>
<summary>Changes</summary>

It claimed to dump all sections by default, but this hasn't been true since 2017: https://reviews.llvm.org/D37717

---
Full diff: https://github.com/llvm/llvm-project/pull/81276.diff


1 Files Affected:

- (modified) llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp (+2-2) 


``````````diff
diff --git a/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp b/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
index 559e7a60489834..74dc519c947726 100644
--- a/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
+++ b/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
@@ -845,8 +845,8 @@ 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 verbose mode is specified in
+  // which case all sections are dumped.
 #define HANDLE_DWARF_SECTION(ENUM_NAME, ELF_NAME, CMDLINE_NAME, OPTION)        \
   if (Dump##ENUM_NAME.IsRequested) {                                           \
     DumpType |= DIDT_##ENUM_NAME;                                              \

``````````

</details>


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


More information about the llvm-commits mailing list