Michael137 wrote:

> @Michael137 said:
> 
> > Btw, as a follow-up to this patch should we check that this is compatible 
> > with dsymutil (i.e., running dsymutil --verify)? I suspect it might need a 
> > fixup (given LLDB doesn't even support this tag)

Yup, dsymutil looks good now, thanks

> There's a couple of switches in llvm/lib/DWARFLinker that look like they want 
> a DW_TAG_template_alias:
DependencyTracker::isTypeTableCandidate in DependencyTracker.cpp
AcceleratorRecordsSaver::save in AcceleratorRecordsSaver.cpp. I'm not sure what 
a test for those would look like but I can look into it if you'd like?

Good question, I'm not familiar with these either, but would make sense to 
address these eventually. Though I don't think it's urgent atm since 
`-gtemplate-alias` isn't the default

https://github.com/llvm/llvm-project/pull/87623
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to