[PATCH] D52790: [analyzer][PlistMacroExpansion] New flag to convert macro expansions to events

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 5 11:25:12 PST 2018


Szelethus added a comment.

In https://reviews.llvm.org/D52790#1285039, @NoQ wrote:

> Looks great, let's land?


I'll probably land it after part 5, in order to ease on rebasing.

> Not sure if i already asked - am i understanding correctly that this is a "poor-man's" support for macro expansions for tools that read plists but do not support those new plist macro dictionaries yet?

Correct, same as `notes-as-events`.

> I wonder how expansions of huge macros will look as events.

Quite funny, actually :D Think about assert, it's quite a mouthful. I'll post screenshots when I have more time.


https://reviews.llvm.org/D52790





More information about the cfe-commits mailing list