[clang] 2c3b12b - [clang-format][NFCish] Obey debug settings
Björn Schäpers via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 2 02:50:29 PDT 2023
Author: Björn Schäpers
Date: 2023-08-02T11:50:14+02:00
New Revision: 2c3b12b5418fa4aa3aeb3f2154ae7bedbd1feac8
URL: https://github.com/llvm/llvm-project/commit/2c3b12b5418fa4aa3aeb3f2154ae7bedbd1feac8
DIFF: https://github.com/llvm/llvm-project/commit/2c3b12b5418fa4aa3aeb3f2154ae7bedbd1feac8.diff
LOG: [clang-format][NFCish] Obey debug settings
Differential Revision: https://reviews.llvm.org/D156228
Added:
Modified:
clang/lib/Format/MacroCallReconstructor.cpp
Removed:
################################################################################
diff --git a/clang/lib/Format/MacroCallReconstructor.cpp b/clang/lib/Format/MacroCallReconstructor.cpp
index 1c295477a922ef..cbdd1683c54d1a 100644
--- a/clang/lib/Format/MacroCallReconstructor.cpp
+++ b/clang/lib/Format/MacroCallReconstructor.cpp
@@ -319,30 +319,31 @@ void MacroCallReconstructor::endReconstruction(FormatToken *Token) {
assert(Token->MacroCtx &&
(ActiveExpansions.size() >= Token->MacroCtx->EndOfExpansion));
for (size_t I = 0; I < Token->MacroCtx->EndOfExpansion; ++I) {
-#ifndef NDEBUG
- // Check all remaining tokens but the final closing parenthesis and optional
- // trailing comment were already reconstructed at an inner expansion level.
- for (auto T = ActiveExpansions.back().SpelledI;
- T != ActiveExpansions.back().SpelledE; ++T) {
- FormatToken *Token = T->Tok;
- bool ClosingParen = (std::next(T) == ActiveExpansions.back().SpelledE ||
- std::next(T)->Tok->isTrailingComment()) &&
- !Token->MacroCtx && Token->is(tok::r_paren);
- bool TrailingComment = Token->isTrailingComment();
- bool PreviousLevel =
- Token->MacroCtx &&
- (ActiveExpansions.size() < Token->MacroCtx->ExpandedFrom.size());
- if (!ClosingParen && !TrailingComment && !PreviousLevel)
- llvm::dbgs() << "At token: " << Token->TokenText << "\n";
- // In addition to the following cases, we can also run into this
- // when a macro call had more arguments than expected; in that case,
- // the comma and the remaining tokens in the macro call will potentially
- // end up in the line when we finish the expansion.
- // FIXME: Add the information which arguments are unused, and assert
- // one of the cases below plus reconstructed macro argument tokens.
- // assert(ClosingParen || TrailingComment || PreviousLevel);
- }
-#endif
+ LLVM_DEBUG([&] {
+ // Check all remaining tokens but the final closing parenthesis and
+ // optional trailing comment were already reconstructed at an inner
+ // expansion level.
+ for (auto T = ActiveExpansions.back().SpelledI;
+ T != ActiveExpansions.back().SpelledE; ++T) {
+ FormatToken *Token = T->Tok;
+ bool ClosingParen = (std::next(T) == ActiveExpansions.back().SpelledE ||
+ std::next(T)->Tok->isTrailingComment()) &&
+ !Token->MacroCtx && Token->is(tok::r_paren);
+ bool TrailingComment = Token->isTrailingComment();
+ bool PreviousLevel =
+ Token->MacroCtx &&
+ (ActiveExpansions.size() < Token->MacroCtx->ExpandedFrom.size());
+ if (!ClosingParen && !TrailingComment && !PreviousLevel)
+ llvm::dbgs() << "At token: " << Token->TokenText << "\n";
+ // In addition to the following cases, we can also run into this
+ // when a macro call had more arguments than expected; in that case,
+ // the comma and the remaining tokens in the macro call will
+ // potentially end up in the line when we finish the expansion.
+ // FIXME: Add the information which arguments are unused, and assert
+ // one of the cases below plus reconstructed macro argument tokens.
+ // assert(ClosingParen || TrailingComment || PreviousLevel);
+ }
+ }());
// Handle the remaining open tokens:
// - expand the closing parenthesis, if it exists, including an optional
// trailing comment
More information about the cfe-commits
mailing list