[PATCH] D52742: [analyzer][PlistMacroExpansion] Part 1.: New expand-macros flag

Whisperity via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 9 03:12:27 PDT 2018


whisperity added inline comments.


================
Comment at: test/Analysis/Inputs/expected-plists/plist-macros-with-expansion.cpp.plist:346
+ <array>
+  <string>/home/eumakri/Documents/2codechecker_dev_env/llvm/tools/clang/test/Analysis/plist-macros-with-expansion.cpp</string>
+ </array>
----------------
Same here, as @xazax.hun mentioned. (Damn when I make a checker I'll be careful to ensure my paths don't contain swearwords... 😜 )

Maybe we should carefully analyse what is exactly needed from the plist and throw the rest, just to make the whole file smaller. Plists are pretty bloaty already...


https://reviews.llvm.org/D52742





More information about the cfe-commits mailing list