aaron.ballman accepted this revision. aaron.ballman added a reviewer: clang-language-wg. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM! Adding the language WG in case there are reviews coming from the peanut gallery. Please wait a day before landing just in case. ================ Comment at: clang/lib/Sema/SemaDeclCXX.cpp:107-111 + if (const auto *BD = dyn_cast<BindingDecl>(Decl)) + VD = dyn_cast_if_present<VarDecl>(BD->getDecomposedDecl()); + else + VD = dyn_cast<VarDecl>(Decl); + if (VD) { ---------------- cor3ntin wrote: > Note that https://reviews.llvm.org/D137244 introduces a utility to do that, > but i''d rather avoid having to many dependencies between these patches. So the plan is to land this change, then in D137244 change this usage to the helper function introduced by that patch? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137172/new/ https://reviews.llvm.org/D137172 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits