sammccall added inline comments.

================
Comment at: clang-tools-extra/trunk/clangd/unittests/TypeHierarchyTests.cpp:629
+      AST, Source.point(), /*ResolveLevels=*/1,
+      TypeHierarchyDirection::Children, Index.get(), 
"/clangd-test/TestTU.cpp");
+  ASSERT_TRUE(bool(Result));
----------------
yes, this should be `testPath(TU.Filename)`

(TU.Filename is always unix-style relative path, testPath() makes it an 
OS-style absolute path)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64308



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to