[PATCH] D110911: [analyzer][NFC] Add RangeSet::dump

2021-10-06 Thread Gabor Marton via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGb8f6c85a831f: [analyzer][NFC] Add RangeSet::dump (authored by martong). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110911/new/ https://reviews.llvm.org/D

[PATCH] D110911: [analyzer][NFC] Add RangeSet::dump

2021-10-06 Thread Gabor Marton via Phabricator via cfe-commits
martong added a comment. Thanks for the reivew! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110911/new/ https://reviews.llvm.org/D110911 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://

[PATCH] D110911: [analyzer][NFC] Add RangeSet::dump

2021-10-06 Thread Denys Petrov via Phabricator via cfe-commits
ASDenysPetrov accepted this revision. ASDenysPetrov added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110911/new/ https://reviews.llvm.org/D110911 _

[PATCH] D110911: [analyzer][NFC] Add RangeSet::dump

2021-10-06 Thread Gabor Marton via Phabricator via cfe-commits
martong updated this revision to Diff 377482. martong added a comment. - Rebase - Mark the dump methods with LLVM_DUMP_METHOD - Add Range::dump Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110911/new/ https://reviews.llvm.org/D110911 Files: cl

[PATCH] D110911: [analyzer][NFC] Add RangeSet::dump

2021-10-01 Thread Denys Petrov via Phabricator via cfe-commits
ASDenysPetrov added a comment. In D110911#3035911 , @steakhal wrote: > Please mark both of them `LLVM_DUMP_METHOD`s. This way they will be stripped > from release builds according to their documentation. And `Range::dump` as well. Repository: rG LLV

[PATCH] D110911: [analyzer][NFC] Add RangeSet::dump

2021-10-01 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. Please mark both of them `LLVM_DUMP_METHOD`s. This way they will be stripped from release builds according to their documentation. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110911/new/ https://reviews.llvm.org/D110911

[PATCH] D110911: [analyzer][NFC] Add RangeSet::dump

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