diff options
| author | Richard Smith <richard-llvm@metafoo.co.uk> | 2013-02-07 03:30:24 +0000 |
|---|---|---|
| committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2013-02-07 03:30:24 +0000 |
| commit | 9ce12e36abf44b6373b88610f3c73fbbbdee9b36 (patch) | |
| tree | 252cae5664a20d1ce68c51dc63bf82c4d0dd5ce2 /clang/lib/Frontend/ChainedIncludesSource.cpp | |
| parent | c121b9b796e40f2eec2aabebb4dfcabe2f21537f (diff) | |
| download | bcm5719-llvm-9ce12e36abf44b6373b88610f3c73fbbbdee9b36.tar.gz bcm5719-llvm-9ce12e36abf44b6373b88610f3c73fbbbdee9b36.zip | |
Simplify FindExternalVisibleDeclsByName by making it return a bool indicating
if it found any decls, rather than returning a list of found decls. This
removes a returning-ArrayRef-to-deleted-storage bug from
MultiplexExternalSemaSource (in code not exercised by any of the clang
binaries), reduces the work required in the found-no-decls case with PCH, and
importantly removes the need for DeclContext::lookup to be reentrant.
No functionality change intended!
llvm-svn: 174576
Diffstat (limited to 'clang/lib/Frontend/ChainedIncludesSource.cpp')
| -rw-r--r-- | clang/lib/Frontend/ChainedIncludesSource.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Frontend/ChainedIncludesSource.cpp b/clang/lib/Frontend/ChainedIncludesSource.cpp index 16eb8890d9b..f414f93812e 100644 --- a/clang/lib/Frontend/ChainedIncludesSource.cpp +++ b/clang/lib/Frontend/ChainedIncludesSource.cpp @@ -191,7 +191,7 @@ CXXBaseSpecifier * ChainedIncludesSource::GetExternalCXXBaseSpecifiers(uint64_t Offset) { return getFinalReader().GetExternalCXXBaseSpecifiers(Offset); } -DeclContextLookupResult +bool ChainedIncludesSource::FindExternalVisibleDeclsByName(const DeclContext *DC, DeclarationName Name) { return getFinalReader().FindExternalVisibleDeclsByName(DC, Name); |
