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

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 14 11:16:42 PST 2018


Szelethus updated this revision to Diff 174074.
Szelethus added a comment.

- Added more documentation
- Changed the nullpointer semantics as @xazax.hun pointed out
- Realize that macro arguments themselves can be macros, handle them too.
- Add a bunch more test. The above point actually fixed a wrong macro expansion is `CALL_LAMBDA` that somehow didn't get noticed. Gotta pay more attention when making the test files.


https://reviews.llvm.org/D52795

Files:
  lib/StaticAnalyzer/Core/PlistDiagnostics.cpp
  test/Analysis/Inputs/expected-plists/plist-macros-with-expansion.cpp.plist
  test/Analysis/plist-macros-with-expansion.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52795.174074.patch
Type: text/x-patch
Size: 155736 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181114/015ecabd/attachment-0001.bin>


More information about the cfe-commits mailing list