[llvm] [llvm-objdump] Document selecting symbols containing commas (PR #221619)
Tianyi Chen via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 8 04:47:43 PDT 2026
chenty2333 wrote:
Thank you for the detailed review. I'm sorry that the previous description was too long and that the patch included unnecessary additions. I have updated my email settings as requested.
I have shortened the help text and documentation, removed the redundant example, added the option links, and made the wording consistent.
As this change only concerns documentation and help text, and existing tests cover the relevant shared logic, I have removed the tests added by this PR and left the original tests intact. This keeps tests for mixing options, repeating options, and additional output checks outside the scope of this PR.
The extra `--implicit-check-not` checks came from AI-assisted suggestions, and I did not sufficiently review whether they were necessary. That was my oversight. I have removed them along with the added tests.
https://github.com/llvm/llvm-project/pull/221619
More information about the llvm-commits
mailing list