[Lldb-commits] [PATCH] D64844: [Target] Remove Target::GetScratchClangASTContext

Alex Langford via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jul 17 12:50:15 PDT 2019

xiaobai added a comment.

Yes, I agree that replacing ClangASTContext uses with TypeSystem would be the right thing to do, and it's what I plan on doing next. There are instances where you really do want a ClangASTContext (e.g. in plugins related to clang expression parsing and objc), and so having a convenience function like this means you don't have to cast the result of every call. This is similar to `ObjCLanguageRuntime::Get`. I don't mind abandoning this patch though.

Luckily, like you pointed out, most uses of ClangASTContext are very generalizable so it shouldn't be too bad to change things over.



More information about the lldb-commits mailing list