[Lldb-commits] [PATCH] D72684: [lldb][NFC] Rename ClangASTContext to ClangTypeSystem
Raphael Isemann via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Jan 14 00:46:20 PST 2020
teemperor created this revision.
teemperor added reviewers: LLDB, aprantl, shafik, clayborg.
Herald added subscribers: lldb-commits, JDevlieghere, abidh.
Herald added a project: LLDB.
This commit renames ClangASTContext to ClangTypeSystem to better reflect what this class is actually supposed to do
(implement the TypeSystem interface for Clang). It also gets rid of the very confusing situation that we have both a
`clang::ASTContext` and a `ClangASTContext` in clang (which sometimes causes Clang people to think I'm fiddling
with Clang's ASTContext when I'm actually just doing LLDB work).
I also have plans to potentially have multiple clang::ASTContext instances associated with one ClangASTContext so
the ASTContext naming will then become even more confusing to people.
Repository:
rLLDB LLDB
https://reviews.llvm.org/D72684
Files:
lldb/source/Symbol/ClangASTContext.cpp
lldb/source/Symbol/ClangTypeSystem.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72684.237870.patch
Type: text/x-patch
Size: 0 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200114/0d80f3fc/attachment.bin>
More information about the lldb-commits
mailing list