[PATCH] D56370: [clangd] Add support for type hierarchy (super types only for now)

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 13 01:17:50 PDT 2019


kadircet added a comment.

In D56370#1427075 <https://reviews.llvm.org/D56370#1427075>, @nridge wrote:

> Ok, I filed a Theia issue <https://github.com/theia-ide/theia/issues/4540> about it for now.


Thanks!

In D56370#1427076 <https://reviews.llvm.org/D56370#1427076>, @nridge wrote:

> Are these tests written manually, or with the help or some kind of tool?


Yes unfortunately :(

Thanks for moving the tests!



================
Comment at: clang-tools-extra/test/clangd/type-hierarchy.test:1
+# RUN: clangd -lit-test < %s | FileCheck -strict-whitespace %s
+{"jsonrpc":"2.0","id":0,"method":"initialize","params":{"processId":123,"rootPath":"clangd","capabilities":{},"trace":"off"}}
----------------
why strict-whitespace?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D56370





More information about the cfe-commits mailing list