[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