[PATCH] D126878: [analyzer] Remove NotifyAssumeClients

2022-06-07 Thread Gabor Marton via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG8131ee4c43a8: [analyzer] Remove NotifyAssumeClients (authored by martong). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126878/new/ https://reviews.llvm.or

[PATCH] D126878: [analyzer] Remove NotifyAssumeClients

2022-06-07 Thread Balázs Benics via Phabricator via cfe-commits
steakhal accepted this revision. steakhal added a comment. This revision is now accepted and ready to land. Great! Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126878/new/ https://reviews.llvm.org/D126878 _

[PATCH] D126878: [analyzer] Remove NotifyAssumeClients

2022-06-06 Thread Gabor Marton via Phabricator via cfe-commits
martong added a comment. In D126878#3553184 , @steakhal wrote: > Measure if it changes any reports. Results did not change, we had equal number of reports in all projects. Other stat numbers are also very close. F23339654: stats.html

[PATCH] D126878: [analyzer] Remove NotifyAssumeClients

2022-06-02 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. Measure if it changes any reports. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126878/new/ https://reviews.llvm.org/D126878 ___ cfe-commits mailing list cfe-commits@lists.llvm

[PATCH] D126878: [analyzer] Remove NotifyAssumeClients

2022-06-02 Thread Gabor Marton via Phabricator via cfe-commits
martong created this revision. martong added reviewers: steakhal, NoQ. Herald added subscribers: manas, ASDenysPetrov, gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun. Herald added a reviewer: Szelethus. Herald added a project: