dblaikie added a comment.

In D106615#2949685 <https://reviews.llvm.org/D106615#2949685>, @yonghong-song 
wrote:

> @dblaikie ping. Could you help take a look at the patch?

Yeah, it's on my list.



================
Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:3478-3479
 
+  if (D->hasAttr<BTFTagAttr>())
+    RealDecl->replaceAnnotations(CollectBTFTagAnnotations(D));
+
----------------
Any reason this has to be done with a replace-style API, rather than 
constructed when the DI node is constructed in the first place?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106615

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

Reply via email to