Swift now generates PDBs for debugging on Windows. llvm and lldb
need a language type to properly handle the output emitted by swiftc.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
- Build Status
Buildable 28343 Build 28342: arc lint + arc unit
Event Timeline
include/llvm/DebugInfo/CodeView/CodeView.h | ||
---|---|---|
161 | I would say we should push this further down in case Microsoft adds a new language. Why not follow D's lead and use S? |
Comment Actions
In the absence of guidance from someone at Microsoft, I would suggest not using 0x11, as they have probably already allocated that to some other language. I'd recommend doing what the D language people did and grab 'S'. If VS ever does something with this info, be aware that you will probably want to change which value we use here.
include/llvm/DebugInfo/CodeView/CodeView.h | ||
---|---|---|
165 | I'd either update the existing comment about D or add a new one for Swift. |
Comment Actions
lgtm, but surely you want to add this to the MapDWLangToCVLang in CodeViewDebug.cpp as well, and perhaps add a test for it.
I would say we should push this further down in case Microsoft adds a new language. Why not follow D's lead and use S?