================ @@ -54,6 +54,20 @@ RecordContext *APIRecord::castToRecordContext(const APIRecord *Record) { } } +void RecordContext::stealRecordChain(RecordContext &Other) { + // If we don't have an empty chain append Other's chain into ours. + if (First) + Last->NextInContext = Other.First; ---------------- QuietMisdreavus wrote:
This is the tiniest of nitpicks, but the fact that this only checks `First` but then accesses `Last` (which semantically makes sense but is semantically suspect) feels wrong to me. 😅 https://github.com/llvm/llvm-project/pull/87772 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits