[PATCH] D142354: [analyzer] Create a stub for an std::variant checker

Balázs Benics via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 25 06:45:50 PST 2023


steakhal added a comment.

I would be interested in some of the free-functions dealing with variants, such as `std::visit()`. https://godbolt.org/z/bbocrz4dG
I hope that's also on the radar.

In D142354#4079643 <https://reviews.llvm.org/D142354#4079643>, @Szelethus wrote:

> In D142354#4078450 <https://reviews.llvm.org/D142354#4078450>, @NoQ wrote:
>
>> Have you found a reasonable amount of code that uses `std::variant`, to test your checker on?
>
> Acid <https://github.com/EQMG/Acid> has a few, csv-parser <https://github.com/ashaduri/csv-parser> in one place, there is a couple in config-loader <https://github.com/netcan/config-loader> and cista <https://github.com/felixguendling/cista>, and a surprising amount in userver <https://github.com/userver-framework/userver>. Though its a question how painful it is to set up their dependencies.

I think bitcoin <https://github.com/bitcoin/bitcoin> and qtbase <https://github.com/qt/qtbase> also have some uses for variant.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142354/new/

https://reviews.llvm.org/D142354



More information about the cfe-commits mailing list