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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
