================
@@ -54,6 +54,13 @@ RecordContext *APIRecord::castToRecordContext(const 
APIRecord *Record) {
   }
 }
 
+void RecordContext::stealRecordChain(RecordContext &Other) {
+  First = Other.First;
+  Last = Other.Last;
+  Other.First = nullptr;
+  Other.Last = nullptr;
+}
----------------
daniel-grumberg wrote:
There is no harm in "stealing" an empty record chain, and it's not necessarily 
even semantically wrong, all it indicates is that the current RecordContext 
won't have any children during any traversals. Semantic problems would still be 
caught by having a test.

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

Reply via email to