[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

teemperor added a comment. I changed the commit name from "Add support for D programming language" to "[lldb] Add support for demangling D symbols" when landing (which is more accurate). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110578/new/ h

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

This revision was automatically updated to reflect the committed changes. Closed by commit rG96a735990839: [lldb] Add support for demangling D symbols (authored by ljmf00, committed by teemperor). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110578/

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

dblaikie added a comment. In D110578#3123164 , @ljmf00 wrote: > @dblaikie Maybe you can land this? Nah, might need someone with more lldb context than me - I don't seem to have a clean check-lldb build right now (so don't have a good baseline to validat

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

ljmf00 added a subscriber: dblaikie. ljmf00 added a comment. @dblaikie Maybe you can land this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110578/new/ https://reviews.llvm.org/D110578 ___ lldb-commits

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

ljmf00 added a comment. Since D111414 got merged, @teemperor can you land this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110578/new/ https://reviews.llvm.org/D110578

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

ljmf00 updated this revision to Diff 378255. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110578/new/ https://reviews.llvm.org/D110578 Files: lldb/include/lldb/Core/Mangled.h lldb/source/Core/Mangled.cpp lldb/source/Symbol/Symtab.cpp lldb/u

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

teemperor added a comment. In D110578#3027707 , @ljmf00 wrote: > In D110578#3027692 , @teemperor > wrote: > >> Do you have commit access or should someone land this for you? > > I don't have commit access, althou

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

ljmf00 added a comment. In D110578#3027692 , @teemperor wrote: > Do you have commit access or should someone land this for you? I don't have commit access, although this is a stacked revision, so there are some dependent patches. Repository: rG LLVM

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

teemperor accepted this revision. teemperor added a comment. Do you have commit access or should someone land this for you? Comment at: lldb/include/lldb/Core/Mangled.h:48 +eManglingSchemeRustV0, +eManglingSchemeD }; Nit which can be fixed when landi

[Lldb-commits] [PATCH] D110578: [lldb] Add support for D programming language

JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110578/new/ https://reviews.llvm.org/D110578 ___