kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land.
Thanks, LGTM! ================ Comment at: clang-tools-extra/clangd/ParsedAST.h:80 + const LangOptions &getLangOpts() const { + return getASTContext().getLangOpts(); ---------------- arphaman wrote: > kadircet wrote: > > can we introduce this helper in a new patch, while changing other > > occurrences in clangd? > Sounds good, I'll do that. can you also revert this change? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69543/new/ https://reviews.llvm.org/D69543 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits