curdeius added inline comments.
================ Comment at: clang/lib/Format/WhitespaceManager.cpp:334-337 + if (Changes[i].NewlinesBefore == 0) + Changes[i].Spaces = + std::max(Changes[i].Spaces, + static_cast<int>(Changes[i].Tok->SpacesRequiredBefore)); ---------------- On a second thought, it seems like a workaround only and a real solution would be to fix https://github.com/llvm/llvm-project/issues/53699 that I've just created. It fixes an immediate problem though. I'll add a fixme in a later revision (or when landing) to remove this when the abovementioned bug is solved properly. Then, this problem should be caught by the assertion added later below. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119419/new/ https://reviews.llvm.org/D119419 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
