[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