gribozavr2 added a comment.

The only functional change that I see in this patch is in 
`clang/lib/Tooling/Transformer/Stencil.cpp`. However, I don't understand how 
that change in the (deprecated) selection() stencil can affect other stencils.



================
Comment at: clang/unittests/Tooling/StencilTest.cpp:375
+  double foo(double d);
+  foo(MACRO);)cpp";
+
----------------
"foo(MACRO);" will fail parsing at the top level, it should be within a 
function, I think.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72274/new/

https://reviews.llvm.org/D72274



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to