diff options
author | Nicola Zaghen <nicola.zaghen@imgtec.com> | 2018-05-15 13:30:56 +0000 |
---|---|---|
committer | Nicola Zaghen <nicola.zaghen@imgtec.com> | 2018-05-15 13:30:56 +0000 |
commit | 3538b39ed55e8bcce9a15f6144d605b6843c1cd0 (patch) | |
tree | a9861ff8de0e3e5be9be55407492d38005e663d5 /clang/lib/Format | |
parent | 80827f10a18b91510dc57ff02cf48751bb1498ff (diff) | |
download | bcm5719-llvm-3538b39ed55e8bcce9a15f6144d605b6843c1cd0.tar.gz bcm5719-llvm-3538b39ed55e8bcce9a15f6144d605b6843c1cd0.zip |
[clang] Update uses of DEBUG macro to LLVM_DEBUG.
The DEBUG() macro is very generic so it might clash with other projects.
The renaming was done as follows:
- git grep -l 'DEBUG' | xargs sed -i 's/\bDEBUG\s\?(/LLVM_DEBUG(/g'
- git diff -U0 master | ../clang/tools/clang-format/clang-format-diff.py -i -p1 -style LLVM
Explicitly avoided changing the strings in the clang-format tests.
Differential Revision: https://reviews.llvm.org/D44975
llvm-svn: 332350
Diffstat (limited to 'clang/lib/Format')
-rw-r--r-- | clang/lib/Format/BreakableToken.cpp | 7 | ||||
-rw-r--r-- | clang/lib/Format/ContinuationIndenter.cpp | 75 | ||||
-rw-r--r-- | clang/lib/Format/Format.cpp | 13 | ||||
-rw-r--r-- | clang/lib/Format/SortJavaScriptImports.cpp | 8 | ||||
-rw-r--r-- | clang/lib/Format/TokenAnalyzer.cpp | 10 | ||||
-rw-r--r-- | clang/lib/Format/TokenAnnotator.cpp | 2 | ||||
-rw-r--r-- | clang/lib/Format/UnwrappedLineFormatter.cpp | 12 | ||||
-rw-r--r-- | clang/lib/Format/UnwrappedLineParser.cpp | 4 | ||||
-rw-r--r-- | clang/lib/Format/UsingDeclarationsSorter.cpp | 2 |
9 files changed, 70 insertions, 63 deletions
diff --git a/clang/lib/Format/BreakableToken.cpp b/clang/lib/Format/BreakableToken.cpp index db39a95bb1d..933dce35403 100644 --- a/clang/lib/Format/BreakableToken.cpp +++ b/clang/lib/Format/BreakableToken.cpp @@ -67,8 +67,9 @@ static BreakableToken::Split getCommentSplit(StringRef Text, unsigned ColumnLimit, unsigned TabWidth, encoding::Encoding Encoding) { - DEBUG(llvm::dbgs() << "Comment split: \"" << Text << ", " << ColumnLimit - << "\", Content start: " << ContentStartColumn << "\n"); + LLVM_DEBUG(llvm::dbgs() << "Comment split: \"" << Text << ", " << ColumnLimit + << "\", Content start: " << ContentStartColumn + << "\n"); if (ColumnLimit <= ContentStartColumn + 1) return BreakableToken::Split(StringRef::npos, 0); @@ -424,7 +425,7 @@ BreakableBlockComment::BreakableBlockComment( } } - DEBUG({ + LLVM_DEBUG({ llvm::dbgs() << "IndentAtLineBreak " << IndentAtLineBreak << "\n"; llvm::dbgs() << "DelimitersOnNewline " << DelimitersOnNewline << "\n"; for (size_t i = 0; i < Lines.size(); ++i) { diff --git a/clang/lib/Format/ContinuationIndenter.cpp b/clang/lib/Format/ContinuationIndenter.cpp index 1d04ad0a499..20eb5c03b8b 100644 --- a/clang/lib/Format/ContinuationIndenter.cpp +++ b/clang/lib/Format/ContinuationIndenter.cpp @@ -1768,12 +1768,12 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, Token->adaptStartOfLine(0, Whitespaces); unsigned Penalty = 0; - DEBUG(llvm::dbgs() << "Breaking protruding token at column " << StartColumn - << ".\n"); + LLVM_DEBUG(llvm::dbgs() << "Breaking protruding token at column " + << StartColumn << ".\n"); for (unsigned LineIndex = 0, EndIndex = Token->getLineCount(); LineIndex != EndIndex; ++LineIndex) { - DEBUG(llvm::dbgs() << " Line: " << LineIndex << " (Reflow: " << Reflow - << ")\n"); + LLVM_DEBUG(llvm::dbgs() + << " Line: " << LineIndex << " (Reflow: " << Reflow << ")\n"); NewBreakBefore = false; // If we did reflow the previous line, we'll try reflowing again. Otherwise // we'll start reflowing if the current line is broken or whitespace is @@ -1781,11 +1781,11 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, bool TryReflow = Reflow; // Break the current token until we can fit the rest of the line. while (ContentStartColumn + RemainingTokenColumns > ColumnLimit) { - DEBUG(llvm::dbgs() << " Over limit, need: " - << (ContentStartColumn + RemainingTokenColumns) - << ", space: " << ColumnLimit - << ", reflown prefix: " << ContentStartColumn - << ", offset in line: " << TailOffset << "\n"); + LLVM_DEBUG(llvm::dbgs() << " Over limit, need: " + << (ContentStartColumn + RemainingTokenColumns) + << ", space: " << ColumnLimit + << ", reflown prefix: " << ContentStartColumn + << ", offset in line: " << TailOffset << "\n"); // If the current token doesn't fit, find the latest possible split in the // current line so that breaking at it will be under the column limit. // FIXME: Use the earliest possible split while reflowing to correctly @@ -1800,7 +1800,7 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, // The last line's penalty is handled in addNextStateToQueue(). Penalty += Style.PenaltyExcessCharacter * (ContentStartColumn + RemainingTokenColumns - ColumnLimit); - DEBUG(llvm::dbgs() << " No break opportunity.\n"); + LLVM_DEBUG(llvm::dbgs() << " No break opportunity.\n"); break; } assert(Split.first != 0); @@ -1827,7 +1827,7 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, // ^--------------- to next split columns unsigned ToSplitColumns = Token->getRangeLength( LineIndex, TailOffset, Split.first, ContentStartColumn); - DEBUG(llvm::dbgs() << " ToSplit: " << ToSplitColumns << "\n"); + LLVM_DEBUG(llvm::dbgs() << " ToSplit: " << ToSplitColumns << "\n"); BreakableToken::Split NextSplit = Token->getSplit( LineIndex, TailOffset + Split.first + Split.second, ColumnLimit, @@ -1847,9 +1847,10 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, // unbreakable sequence. ToNextSplitColumns = Token->getLengthAfterCompression(ToNextSplitColumns, Split); - DEBUG(llvm::dbgs() << " ContentStartColumn: " << ContentStartColumn - << "\n"); - DEBUG(llvm::dbgs() << " ToNextSplit: " << ToNextSplitColumns << "\n"); + LLVM_DEBUG(llvm::dbgs() + << " ContentStartColumn: " << ContentStartColumn << "\n"); + LLVM_DEBUG(llvm::dbgs() + << " ToNextSplit: " << ToNextSplitColumns << "\n"); // If the whitespace compression makes us fit, continue on the current // line. bool ContinueOnLine = @@ -1861,16 +1862,16 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, ExcessCharactersPenalty = (ContentStartColumn + ToNextSplitColumns - ColumnLimit) * Style.PenaltyExcessCharacter; - DEBUG(llvm::dbgs() - << " Penalty excess: " << ExcessCharactersPenalty - << "\n break : " << NewBreakPenalty << "\n"); + LLVM_DEBUG(llvm::dbgs() + << " Penalty excess: " << ExcessCharactersPenalty + << "\n break : " << NewBreakPenalty << "\n"); if (ExcessCharactersPenalty < NewBreakPenalty) { Exceeded = true; ContinueOnLine = true; } } if (ContinueOnLine) { - DEBUG(llvm::dbgs() << " Continuing on line...\n"); + LLVM_DEBUG(llvm::dbgs() << " Continuing on line...\n"); // The current line fits after compressing the whitespace - reflow // the next line into it if possible. TryReflow = true; @@ -1886,7 +1887,7 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, continue; } } - DEBUG(llvm::dbgs() << " Breaking...\n"); + LLVM_DEBUG(llvm::dbgs() << " Breaking...\n"); ContentStartColumn = Token->getContentStartColumn(LineIndex, /*Break=*/true); unsigned NewRemainingTokenColumns = Token->getRemainingLength( @@ -1902,8 +1903,8 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, } assert(NewRemainingTokenColumns < RemainingTokenColumns); - DEBUG(llvm::dbgs() << " Breaking at: " << TailOffset + Split.first - << ", " << Split.second << "\n"); + LLVM_DEBUG(llvm::dbgs() << " Breaking at: " << TailOffset + Split.first + << ", " << Split.second << "\n"); if (!DryRun) Token->insertBreak(LineIndex, TailOffset, Split, Whitespaces); @@ -1941,11 +1942,12 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, // the next logical line. BreakableToken::Split SplitBeforeNext = Token->getReflowSplit(NextLineIndex, CommentPragmasRegex); - DEBUG(llvm::dbgs() << " Size of reflown text: " << ContentStartColumn - << "\n Potential reflow split: "); + LLVM_DEBUG(llvm::dbgs() + << " Size of reflown text: " << ContentStartColumn + << "\n Potential reflow split: "); if (SplitBeforeNext.first != StringRef::npos) { - DEBUG(llvm::dbgs() << SplitBeforeNext.first << ", " - << SplitBeforeNext.second << "\n"); + LLVM_DEBUG(llvm::dbgs() << SplitBeforeNext.first << ", " + << SplitBeforeNext.second << "\n"); TailOffset = SplitBeforeNext.first + SplitBeforeNext.second; // If the rest of the next line fits into the current line below the // column limit, we can safely reflow. @@ -1953,11 +1955,12 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, NextLineIndex, TailOffset, ContentStartColumn); Reflow = true; if (ContentStartColumn + RemainingTokenColumns > ColumnLimit) { - DEBUG(llvm::dbgs() << " Over limit after reflow, need: " - << (ContentStartColumn + RemainingTokenColumns) - << ", space: " << ColumnLimit - << ", reflown prefix: " << ContentStartColumn - << ", offset in line: " << TailOffset << "\n"); + LLVM_DEBUG(llvm::dbgs() + << " Over limit after reflow, need: " + << (ContentStartColumn + RemainingTokenColumns) + << ", space: " << ColumnLimit + << ", reflown prefix: " << ContentStartColumn + << ", offset in line: " << TailOffset << "\n"); // If the whole next line does not fit, try to find a point in // the next line at which we can break so that attaching the part // of the next line to that break point onto the current line is @@ -1966,7 +1969,7 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, Token->getSplit(NextLineIndex, TailOffset, ColumnLimit, ContentStartColumn, CommentPragmasRegex); if (Split.first == StringRef::npos) { - DEBUG(llvm::dbgs() << " Did not find later break\n"); + LLVM_DEBUG(llvm::dbgs() << " Did not find later break\n"); Reflow = false; } else { // Check whether the first split point gets us below the column @@ -1975,9 +1978,9 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, unsigned ToSplitColumns = Token->getRangeLength( NextLineIndex, TailOffset, Split.first, ContentStartColumn); if (ContentStartColumn + ToSplitColumns > ColumnLimit) { - DEBUG(llvm::dbgs() << " Next split protrudes, need: " - << (ContentStartColumn + ToSplitColumns) - << ", space: " << ColumnLimit); + LLVM_DEBUG(llvm::dbgs() << " Next split protrudes, need: " + << (ContentStartColumn + ToSplitColumns) + << ", space: " << ColumnLimit); unsigned ExcessCharactersPenalty = (ContentStartColumn + ToSplitColumns - ColumnLimit) * Style.PenaltyExcessCharacter; @@ -1988,7 +1991,7 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, } } } else { - DEBUG(llvm::dbgs() << "not found.\n"); + LLVM_DEBUG(llvm::dbgs() << "not found.\n"); } } if (!Reflow) { @@ -2030,7 +2033,7 @@ ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, BreakableToken::Split SplitAfterLastLine = Token->getSplitAfterLastLine(TailOffset); if (SplitAfterLastLine.first != StringRef::npos) { - DEBUG(llvm::dbgs() << "Replacing whitespace after last line.\n"); + LLVM_DEBUG(llvm::dbgs() << "Replacing whitespace after last line.\n"); if (!DryRun) Token->replaceWhitespaceAfterLastLine(TailOffset, SplitAfterLastLine, Whitespaces); diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp index 53db932a5a0..f3681a715e4 100644 --- a/clang/lib/Format/Format.cpp +++ b/clang/lib/Format/Format.cpp @@ -930,9 +930,9 @@ std::error_code parseConfiguration(StringRef Text, FormatStyle *Style) { // Ensure that each language is configured at most once. for (unsigned j = 0; j < i; ++j) { if (Styles[i].Language == Styles[j].Language) { - DEBUG(llvm::dbgs() - << "Duplicate languages in the config file on positions " << j - << " and " << i << "\n"); + LLVM_DEBUG(llvm::dbgs() + << "Duplicate languages in the config file on positions " + << j << " and " << i << "\n"); return make_error_code(ParseError::Error); } } @@ -2146,7 +2146,7 @@ llvm::Expected<FormatStyle> getStyle(StringRef StyleName, StringRef FileName, SmallString<128> ConfigFile(Directory); llvm::sys::path::append(ConfigFile, ".clang-format"); - DEBUG(llvm::dbgs() << "Trying " << ConfigFile << "...\n"); + LLVM_DEBUG(llvm::dbgs() << "Trying " << ConfigFile << "...\n"); Status = FS->status(ConfigFile.str()); bool FoundConfigFile = @@ -2155,7 +2155,7 @@ llvm::Expected<FormatStyle> getStyle(StringRef StyleName, StringRef FileName, // Try _clang-format too, since dotfiles are not commonly used on Windows. ConfigFile = Directory; llvm::sys::path::append(ConfigFile, "_clang-format"); - DEBUG(llvm::dbgs() << "Trying " << ConfigFile << "...\n"); + LLVM_DEBUG(llvm::dbgs() << "Trying " << ConfigFile << "...\n"); Status = FS->status(ConfigFile.str()); FoundConfigFile = Status && (Status->getType() == llvm::sys::fs::file_type::regular_file); @@ -2177,7 +2177,8 @@ llvm::Expected<FormatStyle> getStyle(StringRef StyleName, StringRef FileName, return make_string_error("Error reading " + ConfigFile + ": " + ec.message()); } - DEBUG(llvm::dbgs() << "Using configuration file " << ConfigFile << "\n"); + LLVM_DEBUG(llvm::dbgs() + << "Using configuration file " << ConfigFile << "\n"); return Style; } } diff --git a/clang/lib/Format/SortJavaScriptImports.cpp b/clang/lib/Format/SortJavaScriptImports.cpp index e8df04c8bd4..2ec577382ff 100644 --- a/clang/lib/Format/SortJavaScriptImports.cpp +++ b/clang/lib/Format/SortJavaScriptImports.cpp @@ -188,9 +188,9 @@ public: if (FirstNonImportLine && FirstNonImportLine->First->NewlinesBefore < 2) ReferencesText += "\n"; - DEBUG(llvm::dbgs() << "Replacing imports:\n" - << getSourceText(InsertionPoint) << "\nwith:\n" - << ReferencesText << "\n"); + LLVM_DEBUG(llvm::dbgs() << "Replacing imports:\n" + << getSourceText(InsertionPoint) << "\nwith:\n" + << ReferencesText << "\n"); auto Err = Result.add(tooling::Replacement( Env.getSourceManager(), CharSourceRange::getCharRange(InsertionPoint), ReferencesText)); @@ -307,7 +307,7 @@ private: FirstNonImportLine = nullptr; AnyImportAffected = AnyImportAffected || Line->Affected; Reference.Range.setEnd(LineEnd->Tok.getEndLoc()); - DEBUG({ + LLVM_DEBUG({ llvm::dbgs() << "JsModuleReference: {" << "is_export: " << Reference.IsExport << ", cat: " << Reference.Category diff --git a/clang/lib/Format/TokenAnalyzer.cpp b/clang/lib/Format/TokenAnalyzer.cpp index ac31f403d5f..99fc61ef1c3 100644 --- a/clang/lib/Format/TokenAnalyzer.cpp +++ b/clang/lib/Format/TokenAnalyzer.cpp @@ -55,12 +55,12 @@ TokenAnalyzer::TokenAnalyzer(const Environment &Env, const FormatStyle &Style) UnwrappedLines(1), Encoding(encoding::detectEncoding( Env.getSourceManager().getBufferData(Env.getFileID()))) { - DEBUG( + LLVM_DEBUG( llvm::dbgs() << "File encoding: " << (Encoding == encoding::Encoding_UTF8 ? "UTF8" : "unknown") << "\n"); - DEBUG(llvm::dbgs() << "Language: " << getLanguageName(Style.Language) - << "\n"); + LLVM_DEBUG(llvm::dbgs() << "Language: " << getLanguageName(Style.Language) + << "\n"); } std::pair<tooling::Replacements, unsigned> TokenAnalyzer::process() { @@ -74,7 +74,7 @@ std::pair<tooling::Replacements, unsigned> TokenAnalyzer::process() { assert(UnwrappedLines.rbegin()->empty()); unsigned Penalty = 0; for (unsigned Run = 0, RunE = UnwrappedLines.size(); Run + 1 != RunE; ++Run) { - DEBUG(llvm::dbgs() << "Run " << Run << "...\n"); + LLVM_DEBUG(llvm::dbgs() << "Run " << Run << "...\n"); SmallVector<AnnotatedLine *, 16> AnnotatedLines; TokenAnnotator Annotator(Style, Tokens.getKeywords()); @@ -86,7 +86,7 @@ std::pair<tooling::Replacements, unsigned> TokenAnalyzer::process() { std::pair<tooling::Replacements, unsigned> RunResult = analyze(Annotator, AnnotatedLines, Tokens); - DEBUG({ + LLVM_DEBUG({ llvm::dbgs() << "Replacements for run " << Run << ":\n"; for (tooling::Replacements::const_iterator I = RunResult.first.begin(), E = RunResult.first.end(); diff --git a/clang/lib/Format/TokenAnnotator.cpp b/clang/lib/Format/TokenAnnotator.cpp index ef763319d00..21e51dd47e7 100644 --- a/clang/lib/Format/TokenAnnotator.cpp +++ b/clang/lib/Format/TokenAnnotator.cpp @@ -2143,7 +2143,7 @@ void TokenAnnotator::calculateFormattingInformation(AnnotatedLine &Line) { ++IndentLevel; } - DEBUG({ printDebugInfo(Line); }); + LLVM_DEBUG({ printDebugInfo(Line); }); } void TokenAnnotator::calculateUnbreakableTailLengths(AnnotatedLine &Line) { diff --git a/clang/lib/Format/UnwrappedLineFormatter.cpp b/clang/lib/Format/UnwrappedLineFormatter.cpp index 33163f7d0a1..906dae40cbe 100644 --- a/clang/lib/Format/UnwrappedLineFormatter.cpp +++ b/clang/lib/Format/UnwrappedLineFormatter.cpp @@ -905,7 +905,8 @@ private: Penalty = Queue.top().first.first; StateNode *Node = Queue.top().second; if (!Node->State.NextToken) { - DEBUG(llvm::dbgs() << "\n---\nPenalty for line: " << Penalty << "\n"); + LLVM_DEBUG(llvm::dbgs() + << "\n---\nPenalty for line: " << Penalty << "\n"); break; } Queue.pop(); @@ -929,7 +930,7 @@ private: if (Queue.empty()) { // We were unable to find a solution, do nothing. // FIXME: Add diagnostic? - DEBUG(llvm::dbgs() << "Could not find a solution.\n"); + LLVM_DEBUG(llvm::dbgs() << "Could not find a solution.\n"); return 0; } @@ -937,8 +938,9 @@ private: if (!DryRun) reconstructPath(InitialState, Queue.top().second); - DEBUG(llvm::dbgs() << "Total number of analyzed states: " << Count << "\n"); - DEBUG(llvm::dbgs() << "---\n"); + LLVM_DEBUG(llvm::dbgs() + << "Total number of analyzed states: " << Count << "\n"); + LLVM_DEBUG(llvm::dbgs() << "---\n"); return Penalty; } @@ -980,7 +982,7 @@ private: formatChildren(State, (*I)->NewLine, /*DryRun=*/false, Penalty); Penalty += Indenter->addTokenToState(State, (*I)->NewLine, false); - DEBUG({ + LLVM_DEBUG({ printLineState((*I)->Previous->State); if ((*I)->NewLine) { llvm::dbgs() << "Penalty for placing " diff --git a/clang/lib/Format/UnwrappedLineParser.cpp b/clang/lib/Format/UnwrappedLineParser.cpp index c4d92d42d82..f1b0f85eacf 100644 --- a/clang/lib/Format/UnwrappedLineParser.cpp +++ b/clang/lib/Format/UnwrappedLineParser.cpp @@ -260,7 +260,7 @@ void UnwrappedLineParser::parse() { IndexedTokenSource TokenSource(AllTokens); Line->FirstStartColumn = FirstStartColumn; do { - DEBUG(llvm::dbgs() << "----\n"); + LLVM_DEBUG(llvm::dbgs() << "----\n"); reset(); Tokens = &TokenSource; TokenSource.reset(); @@ -2324,7 +2324,7 @@ LLVM_ATTRIBUTE_UNUSED static void printDebugInfo(const UnwrappedLine &Line, void UnwrappedLineParser::addUnwrappedLine() { if (Line->Tokens.empty()) return; - DEBUG({ + LLVM_DEBUG({ if (CurrentLines == &Lines) printDebugInfo(*Line); }); diff --git a/clang/lib/Format/UsingDeclarationsSorter.cpp b/clang/lib/Format/UsingDeclarationsSorter.cpp index a380f28f386..9e49e791303 100644 --- a/clang/lib/Format/UsingDeclarationsSorter.cpp +++ b/clang/lib/Format/UsingDeclarationsSorter.cpp @@ -161,7 +161,7 @@ void endUsingDeclarationBlock( StringRef Text(SourceMgr.getCharacterData(SortedBegin), SourceMgr.getCharacterData(SortedEnd) - SourceMgr.getCharacterData(SortedBegin)); - DEBUG({ + LLVM_DEBUG({ StringRef OldText(SourceMgr.getCharacterData(Begin), SourceMgr.getCharacterData(End) - SourceMgr.getCharacterData(Begin)); |