[PATCH] D93223: [analyzer] Create MacroExpansionContext member in AnalysisConsumer

2021-02-22 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG7c58fb6ba04e: [analyzer] Create MacroExpansionContext member in AnalysisConsumer (authored by steakhal, committed by Balazs Benics ). Repository: rG LLVM Github Monorep

[PATCH] D93223: [analyzer] Create MacroExpansionContext member in AnalysisConsumer

2021-02-11 Thread Balázs Benics via Phabricator via cfe-commits
steakhal updated this revision to Diff 323024. steakhal added a comment. Nothing changed. Rebased on top of 2407eb08a5748bc2613e95fa449fc1cae6f4ff8f . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93223/new/ https://r

[PATCH] D93223: [analyzer] Create MacroExpansionContext member in AnalysisConsumer

2021-02-10 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. In D93223#2553053 , @Szelethus wrote: > My no1. thought is that I wish the new functionality you're implementing was > in the interface of the `Preprocessor`. I found, and still find it so hard to > believe that you couldn't jus

[PATCH] D93223: [analyzer] Create MacroExpansionContext member in AnalysisConsumer

2021-02-09 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus accepted this revision. Szelethus added a comment. Herald added a subscriber: nullptr.cpp. My no1. thought is that I wish the new functionality you're implementing was in the interface of the `Preprocessor`. I found, and still find it so hard to believe that you couldn't just retrieve

[PATCH] D93223: [analyzer] Create MacroExpansionContext member in AnalysisConsumer

2021-02-01 Thread Gabor Marton via Phabricator via cfe-commits
martong accepted this revision. martong added a comment. This revision is now accepted and ready to land. Still looks good. (The parent is an NFC and only this patch changes the behavior (and only if the cmdline flag is there), right? So, I'd expect the big impact from this patch.) CHANGES SIN

[PATCH] D93223: [analyzer] Create MacroExpansionContext member in AnalysisConsumer

2021-01-22 Thread Balázs Benics via Phabricator via cfe-commits
steakhal updated this revision to Diff 318482. steakhal retitled this revision from "[RFC][analyzer] Create MacroExpansionContext member in AnalysisConsumer and pass down to the diagnostics consumers" to "[analyzer] Create MacroExpansionContext member in AnalysisConsumer". steakhal edited the su