This is an archive of the discontinued LLVM Phabricator instance.

Add support for the Dylan language to ClangASTContext
ClosedPublic

Authored by housel on Nov 25 2018, 9:42 PM.

Details

Summary

This change adds eLanguageTypeDylan to the set of languages supported
by ClangASTContext. Debug info generated by the Open Dylan compiler's
LLVM back-end was designed to be compatible with C debug info.

Diff Detail

Repository
rL LLVM

Event Timeline

housel created this revision.Nov 25 2018, 9:42 PM
brucem added a subscriber: lldb-commits.
clayborg accepted this revision.Nov 26 2018, 7:31 AM
This revision is now accepted and ready to land.Nov 26 2018, 7:31 AM
aprantl added inline comments.Nov 26 2018, 9:16 AM
source/Symbol/ClangASTContext.cpp
123 ↗(On Diff #175196)

Please add a comment explaining that "The Debug info generated by the Open Dylan compiler's
LLVM back-end was designed to be compatible with C debug info."

housel updated this revision to Diff 175385.Nov 26 2018, 8:16 PM

Adds an inline comment about Open Dylan debug info

I don't have commit access; could someone land this please? Thanks.

brucem added a subscriber: brucem.Nov 26 2018, 8:21 PM

I can land this today.

housel updated this revision to Diff 175391.Nov 26 2018, 8:47 PM

Updated against r347619

This revision was automatically updated to reflect the committed changes.