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

[Artem Dergachev via Phabricator via cfe-commits]

NoQ added inline comments.


================
Comment at: lib/StaticAnalyzer/Core/AnalyzerOptions.cpp:471
+    DisplayMacroExpansions =
+        getBooleanOption("expand-macros", /*Default=*/false);
+  return DisplayMacroExpansions.getValue();
----------------
Yup, np, let's wait until it lands then.


================
Comment at: test/Analysis/Inputs/expected-plists/plist-macros-with-expansion.cpp.plist:17-18
+     </dict>
+     <key>name</key><string></string>
+     <key>expansion</key><string></string>
+    </dict>
----------------
They look empty. I don't think this is intended.


https://reviews.llvm.org/D52742