In Apple's downstream fork, there is support for understanding the swift
AST sections in various binaries. Even though the lldb on llvm.org does
not have support for debugging swift, I think it makes sense to move
support for recognizing swift ast sections upstream.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
lldb/source/Core/Section.cpp | ||
---|---|---|
153 | I wonder if this should be "swiftmodules". I have never seen it spelled with a hyphen. |
LGTM with @kastiglione's comments addressed.
lldb/source/Core/Section.cpp | ||
---|---|---|
153 | Is it actually more than one module? If not, swift-module would make sense given the DWARF cases - it is a "Swift Module" converted to lower kebab case. |
lldb/source/Core/Section.cpp | ||
---|---|---|
153 | I added a hyphen to match the previous section types, but I'm not tied to the naming scheme. |
It seems like there is consensus for actually implementing this functionality but that there were some questions about the actual string. I hope everyone finds my reasoning understandable. If nobody has a strong objection to the actual string, I will land this tomorrow morning or afternoon.
I wonder if this should be "swiftmodules". I have never seen it spelled with a hyphen.