sammccall added inline comments.
================ Comment at: clang-tools-extra/clangd/FindTarget.cpp:639 + void VisitDesignatedInitExpr(const DesignatedInitExpr *DIE) { + for (const DesignatedInitExpr::Designator &D : DIE->designators()) { + if (!D.isFieldDesignator()) ---------------- you're breaking after the first one - I think you'd like to report every one instead? You'd test this with a DIE like `{ .Foo.Bar = 2 }` where `Foo` has struct type. (targetDecl only reports the *last* one, because the Designator can't be a DynTypedNode, but we don't care about that here) ================ Comment at: clang-tools-extra/clangd/unittests/FindTargetTests.cpp:566 TU.ExtraArgs.push_back("-fno-delayed-template-parsing"); - TU.ExtraArgs.push_back("-std=c++17"); + TU.ExtraArgs.push_back("-std=c++2a"); ---------------- isn't this spelled c++20 yet? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72867/new/ https://reviews.llvm.org/D72867 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits