MyDeveloperDay added inline comments.
================ Comment at: clang/lib/Format/MacroExpander.cpp:150 + Tok->MacroCtx.ExpandedFrom.push_back(ID); + if (First) { + Tok->MacroCtx.StartOfExpansion = true; ---------------- elide braces? ================ Comment at: clang/unittests/Format/MacroExpanderTest.cpp:5 + +#include "gmock/gmock.h" +#include "gtest/gtest.h" ---------------- are you using this? ================ Comment at: clang/unittests/Format/MacroExpanderTest.cpp:55 + .str(); + EXPECT_TRUE(Tokens[I]->is(Attributes[I].Kind)) + << Context << " in " << text(Tokens); ---------------- when these assertions fail you have no idea which of the various calls is actually failing how about passing in __FILE__,__LINE__ then adding that to the output Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83296/new/ https://reviews.llvm.org/D83296 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits