[clang-tools-extra] [clangd][RFC] Add container field to remote index Refs grpc method (PR #71605)

kadir çetinkaya via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 4 07:28:42 PST 2023


================
@@ -223,6 +223,7 @@ TEST(RemoteMarshallingTest, RefSerialization) {
   Location.FileURI = testPathURI(
       "llvm-project/llvm/clang-tools-extra/clangd/Protocol.h", Strings);
   Ref.Location = Location;
+  Ref.Container = llvm::cantFail(SymbolID::fromStr("0000000000000001"));
----------------
kadircet wrote:

this is actually not being tested properly, can you update https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/clangd/index/YAMLSerialization.cpp#L318 to serialize container to YAML ?

https://github.com/llvm/llvm-project/pull/71605


More information about the cfe-commits mailing list