Skip to content

Conversation

@keith
Copy link
Member

@keith keith commented Nov 14, 2025

This was previously under the ELF specific options section, but is
actually only supported for Mach-O

@llvmbot
Copy link
Member

llvmbot commented Nov 14, 2025

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

Author: Keith Smiley (keith)

Changes

This was previously under the ELF specific options section, but is
actually only supported for Mach-O


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

1 Files Affected:

  • (modified) llvm/docs/CommandGuide/llvm-strip.rst (+7)
diff --git a/llvm/docs/CommandGuide/llvm-strip.rst b/llvm/docs/CommandGuide/llvm-strip.rst
index b4d6dc08dc1ac..61527e468e19c 100644
--- a/llvm/docs/CommandGuide/llvm-strip.rst
+++ b/llvm/docs/CommandGuide/llvm-strip.rst
@@ -179,6 +179,13 @@ them.
  segments. Note that many tools will not be able to use an object without
  section headers.
 
+MACH-O-SPECIFIC OPTIONS
+--------------------
+
+The following options are implemented only for Mach-O objects. If used with other
+objects, :program:`llvm-strip` will either emit an error or silently ignore
+them.
+
 .. option:: -T
 
  Remove Swift symbols.

Copy link
Collaborator

@jh7370 jh7370 left a comment

Choose a reason for hiding this comment

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

LGTM, with one fix.

I note from the pre-merge checks that in addition to the issue I pointed out inline, there's an unrelated error in the docs build. Would you mind addressing (in a separate PR) while tweaking docs?

error: invalid header in 'CommandGuide/llvm-ir2vec.rst' (does not match title)

(I think this means the underline and the text for the leading header in that file mismatch)

This was previously under the ELF specific options section, but is
actually only supported for Mach-O
@keith keith force-pushed the ks/docs-fix-llvm-strip-t-flag-section branch from 2fcc370 to fac5b2e Compare November 14, 2025 18:23
@keith
Copy link
Member Author

keith commented Nov 14, 2025

submitted #167987 with that fix

@keith keith merged commit 9fcb675 into llvm:main Nov 14, 2025
9 of 10 checks passed
@keith keith deleted the ks/docs-fix-llvm-strip-t-flag-section branch November 14, 2025 19:07
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