[PATCH] D52742: [analyzer][WIP] Add macro expansions to the plist output

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 2 10:04:51 PDT 2018


Szelethus added inline comments.


================
Comment at: lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:755
+        if (It->is(tok::l_paren)) {
+          while ((++It)->isNot(tok::r_paren)) {
+            assert(It->isNot(tok::eof) &&
----------------
donat.nagy wrote:
> I fear that this does not handle nested parentheses correctly; e.g. in a case like `MACRO_ONE(foo, MACRO_TWO(bar, baz), spam)`, it does not skip `, spam)`.
And rightfully so, nice catch!


https://reviews.llvm.org/D52742





More information about the cfe-commits mailing list