[PATCH] D89571: [clangd] Add textDocument/ast extension method to dump the AST

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 19 23:47:23 PST 2020


kadircet added inline comments.


================
Comment at: clang-tools-extra/clangd/unittests/DumpASTTests.cpp:12
+#include "TestTU.h"
+#include "llvm/Support/ScopedPrinter.h"
+#include "gmock/gmock.h"
----------------
sammccall wrote:
> adamcz wrote:
> > include order lint warning here
> Not sure what's going on here, this does seem to be the order that current clang-format wants. Maybe this clang-tidy is an older version?
Has been fixed with https://github.com/llvm/llvm-project/commit/5a9f3867046c4e1c97760e22a505f4d1d788417e


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89571/new/

https://reviews.llvm.org/D89571



More information about the cfe-commits mailing list