[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.
More information about the cfe-commits