Skip to content

Conversation

@JDevlieghere
Copy link
Member

Gently discourage users from relying on -flat by specifying in the help output that it's meant for testing.

Gently discourage users from relying on -flat by specifying in the help
output that it's meant for testing.
@llvmbot
Copy link
Member

llvmbot commented Nov 18, 2025

@llvm/pr-subscribers-debuginfo

Author: Jonas Devlieghere (JDevlieghere)

Changes

Gently discourage users from relying on -flat by specifying in the help output that it's meant for testing.


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

1 Files Affected:

  • (modified) llvm/tools/dsymutil/Options.td (+1-1)
diff --git a/llvm/tools/dsymutil/Options.td b/llvm/tools/dsymutil/Options.td
index e99bc12fa7fd8..571f90c1e46f5 100644
--- a/llvm/tools/dsymutil/Options.td
+++ b/llvm/tools/dsymutil/Options.td
@@ -94,7 +94,7 @@ def: Flag<["-"], "s">,
   Group<grp_general>;
 
 def flat: F<"flat">,
-  HelpText<"Produce a flat dSYM file (not a bundle).">,
+  HelpText<"Produce a flat dSYM file (not a bundle). Intended for testing and generally unsupported by tools that consume dSYMs.">,
   Group<grp_general>;
 def: Flag<["-"], "f">,
   Alias<flat>,

Copy link
Collaborator

@jimingham jimingham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JDevlieghere JDevlieghere enabled auto-merge (squash) November 18, 2025 18:57
@JDevlieghere JDevlieghere merged commit e93763e into llvm:main Nov 18, 2025
9 of 11 checks passed
@JDevlieghere JDevlieghere deleted the dsymutil-flat-help branch November 18, 2025 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants