[PATCH] D127643: [Static Analyzer] Structured bindings to data members

2022-07-06 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. In D127643#3595731 , @steakhal wrote: > I missed this commit because it was not tagged by "[analyzer]". > What is the community preference about tagging CSA commits? I think [analyzer] is a good tag, no need to change anything. I als

[PATCH] D127643: [Static Analyzer] Structured bindings to data members

2022-07-01 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. I read https://en.cppreference.com/w/cpp/language/structured_binding carefully, and there are a number of interesting rules that might deserve their own test case, even if this isn't the patch where you solve that issue, or believe that the solution handles it without

[PATCH] D127643: [Static Analyzer] Structured bindings to data members

2022-06-20 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. I missed this commit because it was not tagged by "[analyzer]". What is the community preference about tagging CSA commits? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127643/new/ https://reviews.llvm.org/D127643 __

[PATCH] D127643: [Static Analyzer] Structured bindings to data members

2022-06-17 Thread Domján Dániel via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe77ac66b8c1c: [Static Analyzer] Structured binding to data members (authored by isuckatcs). Herald added a subscriber: cfe-commits. Repository: rG