Skip to content

Conversation

@madsmtm
Copy link
Contributor

@madsmtm madsmtm commented Oct 30, 2025

It looks like the documentation for llvm-cxxfilt's --[no-]strip-underscore options weren't updated when #106233 was made.

CC @Michael137 (I don't have merge rights myself).

@llvmbot
Copy link
Member

llvmbot commented Oct 30, 2025

@llvm/pr-subscribers-llvm-binary-utilities

Author: Mads Marquart (madsmtm)

Changes

It looks like the documentation for llvm-cxxfilt's --[no-]strip-underscore options weren't updated when #106233 was made.

CC @Michael137 (I don't have merge rights myself).


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

1 Files Affected:

  • (modified) llvm/docs/CommandGuide/llvm-cxxfilt.rst (+2-3)
diff --git a/llvm/docs/CommandGuide/llvm-cxxfilt.rst b/llvm/docs/CommandGuide/llvm-cxxfilt.rst
index 8c61cedd9b70b..8e509cec6ce02 100644
--- a/llvm/docs/CommandGuide/llvm-cxxfilt.rst
+++ b/llvm/docs/CommandGuide/llvm-cxxfilt.rst
@@ -54,8 +54,7 @@ OPTIONS
 
 .. option:: --no-strip-underscore, -n
 
-  Do not strip a leading underscore. This is the default for all platforms
-  except Mach-O based hosts.
+  Do not strip a leading underscore. This is the default for all platforms.
 
 .. option:: --quote
 
@@ -64,7 +63,7 @@ OPTIONS
 .. option:: --strip-underscore, -_
 
   Strip a single leading underscore, if present, from each input name before
-  demangling. On by default on Mach-O based platforms.
+  demangling.
 
 .. option:: --types, -t
 

Copy link
Member

@Michael137 Michael137 left a comment

Choose a reason for hiding this comment

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

Thanks!

@Michael137 Michael137 merged commit a8656c5 into llvm:main Oct 30, 2025
13 checks passed
@madsmtm madsmtm deleted the cxxfilt-docs-update branch October 30, 2025 13:24
aokblast pushed a commit to aokblast/llvm-project that referenced this pull request Oct 30, 2025
It looks like the documentation for `llvm-cxxfilt`'s
`--[no-]strip-underscore` options weren't updated when
llvm#106233 was made.

CC @Michael137 (I don't have merge rights myself).
luciechoi pushed a commit to luciechoi/llvm-project that referenced this pull request Nov 1, 2025
It looks like the documentation for `llvm-cxxfilt`'s
`--[no-]strip-underscore` options weren't updated when
llvm#106233 was made.

CC @Michael137 (I don't have merge rights myself).
DEBADRIBASAK pushed a commit to DEBADRIBASAK/llvm-project that referenced this pull request Nov 3, 2025
It looks like the documentation for `llvm-cxxfilt`'s
`--[no-]strip-underscore` options weren't updated when
llvm#106233 was made.

CC @Michael137 (I don't have merge rights myself).
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