Author: Joachim Meyer Date: 2021-05-10T14:24:18-07:00 New Revision: f3e07c841e2f96a73b253d3b1a95e2ac8df5a376
URL: https://github.com/llvm/llvm-project/commit/f3e07c841e2f96a73b253d3b1a95e2ac8df5a376 DIFF: https://github.com/llvm/llvm-project/commit/f3e07c841e2f96a73b253d3b1a95e2ac8df5a376.diff LOG: [NFC] Correctly assert the indents for printEnumValHelpStr. Only verify that there's no negative indent. Noted by @chapuni in https://reviews.llvm.org/D93494. Reviewed By: chapuni Differential Revision: https://reviews.llvm.org/D102021 (cherry picked from commit d9f2960c932c9803e662098e33d899efa3c67f44) Added: Modified: llvm/lib/Support/CommandLine.cpp Removed: ################################################################################ diff --git a/llvm/lib/Support/CommandLine.cpp b/llvm/lib/Support/CommandLine.cpp index e2f014d1815b7..123a23a5242c2 100644 --- a/llvm/lib/Support/CommandLine.cpp +++ b/llvm/lib/Support/CommandLine.cpp @@ -1729,7 +1729,7 @@ void Option::printHelpStr(StringRef HelpStr, size_t Indent, void Option::printEnumValHelpStr(StringRef HelpStr, size_t BaseIndent, size_t FirstLineIndentedBy) { const StringRef ValHelpPrefix = " "; - assert(BaseIndent >= FirstLineIndentedBy + ValHelpPrefix.size()); + assert(BaseIndent >= FirstLineIndentedBy); std::pair<StringRef, StringRef> Split = HelpStr.split('\n'); outs().indent(BaseIndent - FirstLineIndentedBy) << ArgHelpPrefix << ValHelpPrefix << Split.first << "\n"; _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits