labath added a comment. In D69309#1787297 <https://reviews.llvm.org/D69309#1787297>, @jarin wrote:
> In D69309#1752738 <https://reviews.llvm.org/D69309#1752738>, @friss wrote: > > > Basically, today the debug info will describe an entity named "Foo<int>". > > The accelerator tables all reference this name. So when Clang asks us if we > > know "Foo" (which is what happens when instantiating), we fail to find the > > right instantiations. The consensus of the above discussion was that we > > should change the debug info to have "Foo" as the name of any > > instantiation, with a child DIE describing the template arguments. Just > > doing this in the compiler causes test failures in LLDB, so there's some > > work to do in LLDB to support this. > > > Frederic, you say that "doing this in the compiler causes test failures in > LLDB", which implies you have tried adding the template in the compiler. Do > you have that compiler patch lying around so that we could have a look at > what can be done on the lldb side? > > I agree that a good long term fix is to have "Foo" as an entity in DWARF, > although for backwards compatibility it might be better if the "Foo" template > just contained references to the instantiations rather than having them as > children. I am afraid you're overestimating the scope of that idea. I *think* that Fred was referring to simply changing the string that gets put into the DW_AT_name field of the /instantation/ (and, by extension, the accelerator table). The debug info would still describe instantiations only. I don't believe anyone here was proposing to have DWARF actually describe templates themselves -- that might be possible, but you'd have to get pretty creative with dwarf attributes (and convince a bunch of people that this is actually a good idea). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69309/new/ https://reviews.llvm.org/D69309 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
