sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land.
================ Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:453 const ASTContext &Ctx = Inputs.AST->getASTContext(); + // FIXME: Enable C (and maybe objective-c) once we start spelling types + // explicitly instead of making use of auto. ---------------- I think "non-C++" would be clearer, and don't mention obj-c at all. Formally C++ is independent of whether obj-c is enabled, and that's relevant here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75053/new/ https://reviews.llvm.org/D75053 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits