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

Reply via email to