[PATCH] D52795: [analyzer][PlistMacroExpansion] Part 3.: Macro arguments are expanded

[Gábor Horváth via Phabricator via cfe-commits]

xazax.hun added inline comments.


================
Comment at: lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:685
   const MacroInfo *MI = nullptr;
+  llvm::Optional<MacroArgMap> Args;
 
----------------
I wonder if the optional gives us any value here. An empty map could be just as great to represent that there are no arguments.


https://reviews.llvm.org/D52795